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

Kontonummer in Buchungsliste #71

Closed
JohannMaierhofer opened this issue Nov 4, 2023 · 7 comments
Closed

Kontonummer in Buchungsliste #71

JohannMaierhofer opened this issue Nov 4, 2023 · 7 comments

Comments

@JohannMaierhofer
Copy link

JohannMaierhofer commented Nov 4, 2023

Wenn man mehrere Konten hat und im Fenster Buchungen kein spezielles Konto ausgewählt ist hat man die Buchungen aller Konten. Erzeugt man dann über den Button "PDF Einzelbuchungen" die Buchungsliste erhält man die Buchungen aller Konten.
Leider kann man nicht erkennen von welchem Konto die einzelnen Buchungen sind da nur die Auszug Nummer im Bericht vorkommt und nicht auch das Konto. Das ist in diesem Ausdruck aber dringend nötig.
Ich habe den Code angepasst und das Konto eingefügt mit Anpassung der Spaltenbreiten. Das Konto ist jetzt immer eingefügt. Natürlich könnte man evtl. abfragen (wenn das geht?) ob der Report nur für ein Konto oder mehre ist. Bei nur einem Konto bräuchte man die Kontonummer evtl. nicht.
BuchungAuswertungPDF.zip

@willuhn
Copy link
Member

willuhn commented Nov 6, 2023

Danke, dass du die Änderung direkt selbst vorgenommen hast. Kannst du das aber bitte als Pull-Request einreichen? Dann kann die Änderung von den anderen Mitentwicklern geprüft und übernommen werden.

@JohannMaierhofer
Copy link
Author

Ich bin leider kein erfahrener Programmierer und kenne mich mit Git nicht aus. Da müsste ich erst lernen wie ich einen Pull Request erstellen kann. Ich glaube, so leicht ist das nicht.
Ich habe aus einer Anleitung mir eine Eclipse Umgebung aufgesetzt (noch aus dem alten Repository, Heiners Version). Da konnte ich meine Änderungen machen. In BuchungAuswertungPDF.java gab es im neuen Repository bereits eine Änderung in Zeile 81. Die habe ich jetzt auch gemacht und neu hochgeladen. Die Files in meinem anderen Request waren im neuen Repository noch nicht geändert und sollten darum direkt verwendbar sein.
Ich weiß jetzt nicht wie wir weiter verfahren sollen. Am einfachsten wäre es sicherlich wenn jemand aus eurem Team versucht die Datei zu übernehmen.
In kalten, schlechten Wintertagen werde ich zumindest das neue Repository in Eclipse aufsetzten und mich etwas mit Git beschäftigen. Dann kann ich versuchen auch Pull Requests zu erstellen. Momentan bin ich da etwas überfordert.

dippeal pushed a commit to dippeal/jverein that referenced this issue Nov 6, 2023
@dippeal
Copy link
Member

dippeal commented Nov 6, 2023

Ich habe einen PR #73 erstellt - bitte prüfen.

@willuhn
Copy link
Member

willuhn commented Nov 6, 2023

Danke für das Erstellen des PR. Habe mein Approve gegeben.

@dippeal
Copy link
Member

dippeal commented Nov 6, 2023

Wenn JohannMaierhofer als Mitglied aufgenommen wird, könnte er auch Codes reviews erstellen.

@willuhn
Copy link
Member

willuhn commented Nov 6, 2023

Kann ich gern machen. Dazu muss er aber mit PRs, GIT und Branches umgehen können. Laut seinem Kommentar #71 (comment) fehlt ihm dazu noch die Kenntnis.

willuhn pushed a commit that referenced this issue Nov 12, 2023
* Kontonummer in Buchungsliste #71

#71

* New: Checkbox "Kontonummer in Buchungsliste"

#73 (comment)

* Fix: Add getter & setter, valueOf

---------

Co-authored-by: Alexander Dippe <[email protected]>
@MSchmalzl
Copy link

Erledigt mit #73.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants