Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature Bulk Auflösen #155

Merged
merged 5 commits into from
Feb 18, 2024

Conversation

JohannMaierhofer
Copy link

Nachdem es jetzt ein Bulk Splitbuchung gibt habe ich auch ein Bulk Auflösen implementiert.
Es kann ja sein, dass man bei Bulk Splitbuchung einen Fehler gemacht hat und alle erzeugten Splitbuchungen wieder auflösen möchte. Momentan geht das nur für jede Buchung einzeln über den Splitbuchung View.
Ich habe jetzt ein Contextmenü in der Buchungsliste eingeführt welches bei selektierten Splitbuchungen aktiv ist. Damit lassen sich alle selektierten Splitbuchungen auflösen.
Sonstiges:
Ich habe ein passenderes Icon für das Auflösen gewählt, auch im Splitbuchung View angepasst.
Nach Splitbuchung Auflösen im Splitbuchung View blieb eine Buchung im View sichtbar die aber nicht mehr in der Liste war. Ich habe ein Refresh der Liste eingebaut.
Beim Auflösen der Splitbuchungen wurde die Buchungsliste umsortiert (ein ähnliches Problem wie ich schon früher gesehen hatte). Es lag daran, dass in der Methode refreshBuchungen() in der Datei BuchungsControl.java ein buchungsList.sort() gefehlt hat.
Screenshot_20240216_170149

@MSchmalzl
Copy link

Bei mir fehlt jetzt aber die möglichkeit Bulk-Splitbuchungen zu erstellen.

@JohannMaierhofer
Copy link
Author

Bei mir fehlt jetzt aber die möglichkeit Bulk-Splitbuchungen zu erstellen.

Das sind zwei getrennte Features. Man muss beide übernehmen.

@MSchmalzl
Copy link

Habe es gerade auch gesehen. Machen wir dann erst #153 fertig und aktualisieren dann diesen Branch?

@JohannMaierhofer
Copy link
Author

JohannMaierhofer commented Feb 17, 2024

Habe es gerade auch gesehen. Machen wir dann erst #153 fertig und aktualisieren dann diesen Branch?

Ja, BuchungMenu.java muss man mergen. Alle anderen Dateien sind verschieden.

@dippeal dippeal added the enhancement New feature or request label Feb 17, 2024
@JohannMaierhofer
Copy link
Author

Merge Konflikt ist behoben.

@dippeal dippeal merged commit da4fd83 into openjverein:master Feb 18, 2024
@JohannMaierhofer JohannMaierhofer deleted the feature_bulkaufloesen branch February 18, 2024 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants