-
Notifications
You must be signed in to change notification settings - Fork 15
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/belegnummer #24
Conversation
…tfunktion aus anderen Buchungsfeldern) + neue Einstellmöglichkeit
…erwendeten Einstellung)
…die Belegnummer (anstatt der Buchungsnummer) anzeigen
…nt automatisch pro Konto und pro Geschäftsjahr (de)aktiviert werden.
… erstellten Felder in den Einstellungen wieder gelöscht werden
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code sieht gut aus. Lokaler Test war erfolgreich.
Bei vielen Änderungen handelt es sich nur um Formatierung. Eine einheitliche coding convention für die Entwicklung von openverein halte ich für sinnvoll. Der hier verwendete Style sieht nach Standard Java convention aus. Vielleicht sollte man im kompletten openjverein Code die Umlaute der Variablen ersetzen.
Es gibt einen Fehler: Forum. Das sollte vor einem Merge behoben werden. |
Hallo zusammen, |
Du musst als erstes der OpenJVerein Organisation beitreten. Hierzu einfach ein Issue erstellen. |
Hier nun die im Forum angemerkte, notwendige Korrektur (https://jverein-forum.de/viewtopic.php?p=19526#p19526) in Zeile 238.
Dieser PR hier wurde unter #58 nochmal korrigiert eingereicht. Laut #72 (comment) kann #24 damit gelöscht werden. @kpatzwald OK von deiner Seite? |
@willuhn Sorry, ich bin raus. Nutze die Software nicht mehr und nachdem einige Monate nichts mehr passiert ist, habe ich mich aus dem Projekt ausgetragen und meine lokale Testumgebung gelöscht. Habe zudem keine Zeit mehr. :-( |
OK. Danke für die Rückmeldung. Dann schließe ich den PR. Die Änderungen sind in #58 enthalten. |
So, maybe, I did not do that correct: This is the merge for the open Pull Request by VinRud.
I forget to create a branch first.
We have to test this code very carefully. There are many changes.