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.

Sign up for GitHub

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

Jump to bottom

Neu Button im Sollbuchungen Liste View #402

Merged
merged 6 commits into from
Oct 26, 2024
Merged

Neu Button im Sollbuchungen Liste View #402

merged 6 commits into from
Oct 26, 2024

Conversation

JohannMaierhofer
Copy link

Man kann jetzt auch Sollbuchungen aus dem Sollbuchungen Liste View erzeugen.
Die meisten anderen Views haben das ja auch. Über das Mitgliedskonto im Mitglied View ist das etwas umständlich.

Ich habe ein Input Feld für das Mitglied in den Sollbuchung Detail View eingebaut. Wird Editieren aufgerufen so wird nur das Mitglied angezeigt und es ist nicht ändernbar.
Bei einer neuen Sollbuchung kann man aus der Liste das Mitglied auswählen. Ich habe die Implementierung der Liste aus dem Lesefeld-Definitionen übernommen. Das könnte man in Zukunft verbessern z.B. durch Suchen wie bei Buchungsarten.

@JohannMaierhofer JohannMaierhofer added the enhancement New feature or request label Oct 24, 2024
@JohannMaierhofer
Copy link
Author

PS: Bei Zusatzbeiträgen, Lehrgängen, Wiedervorlage und Arbeiteinsatz plane ich das auch noch.
Arbeitseinsatz und Zusatzbeiträge gibt es auch im Kontextmenü der Mitglieder, aber nicht Sollbuchung, Lehrgänge, Wiedervorlage.
Irgendwie ist das auch inkonsistent.

@JohannMaierhofer JohannMaierhofer marked this pull request as draft October 24, 2024 17:08
@JohannMaierhofer
Copy link
Author

Da ich den Mitgliedinput noch öfters brauche werde ich jetzt doch einen gemeinsamen Input wie bei Buchungsarten machen.

@JohannMaierhofer
Copy link
Author

Jetzt habe ich analog zu Buchungsart einen MitgliedInput und MitgliedSearchInput erzeugt.
Das BuchungBuchungsartAuswahl Enum habe ich in AbstractInputAuswahl umbenannt. Damit kann man es allgemein nutzen.
In den Einstellungen unter Anzeige kann man auswählen ob man die ganze Liste oder die Suche will, so wie auch bei Buchungsart.

Bei der Lesefeld-Definitionen habe ich den Input nicht eingebaut. Da müsste man mehr umbauen da der View nicht ganz unserem Schema entspricht.

@JohannMaierhofer JohannMaierhofer marked this pull request as ready for review October 24, 2024 18:51
lenilsas
lenilsas approved these changes Oct 25, 2024
View reviewed changes
Copy link

@lenilsas lenilsas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Schön!
Ich finde gut, wenn du das auch bei Lehrgägen, Arbeitseinsätzen, Wiedervorlage und Zusatzbeiträgen machst.
Den neuen MitgliedInput könnte man auch bei Mitglied->Vollzahlendes Familienmitglied machen.

@dippeal dippeal merged commit 87265cc into openjverein:master Oct 26, 2024
@JohannMaierhofer JohannMaierhofer deleted the feature_sollbuchung_neu branch October 26, 2024 07:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dippeal dippeal dippeal approved these changes

@lenilsas lenilsas lenilsas approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@JohannMaierhofer @dippeal @lenilsas