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

Fix Spendenbescheinigung Mailversand und Konto Buchungsartauswahl #666

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix Spendenbescheinigung Mailversand und Konto Buchungsartauswahl #666

wants to merge 2 commits into from

Conversation

JohannMaierhofer
Copy link

Der Fix behebt zwei Fehler die im Forum gemeldet wurden:

  • Bei Spendenbescheinigungen wurde in Bei Dateinamen für Spendenbescheinigungen das Spendedatum verwenden #630 im Dateinamen das Spendendatum verwendet. Beim Mailversand wurde es nicht geändert und darum werden die generierten Dateien beim Mailversand nicht gefunden. Der Fix ändert den Dateinamen auch beim Versand
  • Im Konto Dialog gibt es die Inputs für Anlagen Buchungsart und AfA Buchungsart. Wegen des Bugs in Jameica vor der Release 2.10.5 funktioniert dort die Suchauswahl nicht. Der Fix hier frägt die Jameica Version ab und setzt bei alten Jameica Versionen fest die Combobox Auswahl.
    PS: Der Bug in Jameica war, dass nach Disable des Inputs bei Enable kein Editieren mehr möglich ist

Wegen der Fehlers sollten wir wohl eine 3.0.2 herausgeben.

@JohannMaierhofer JohannMaierhofer added the fix This will fix a bug label Feb 13, 2025
lenilsas
lenilsas previously approved these changes Feb 13, 2025
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.

Ich glaube du musst die Version in plugin.xml noch hochsetzen.

@JohannMaierhofer
Copy link
Author

Ich glaube du musst die Version in plugin.xml noch hochsetzen.

Ist gemacht

@tolot27 tolot27 added the blocked Depends on another feature or request label Feb 13, 2025
tolot27
tolot27 approved these changes Feb 13, 2025
View reviewed changes
Copy link
Member

@tolot27 tolot27 left a comment

Choose a reason for hiding this comment

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

#663 sollte zuerst gemerged und unser Java11-Plugin mit dem Java 17 Jameica/SWT getestet werden.

@JohannMaierhofer JohannMaierhofer mentioned this pull request Feb 14, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tolot27 tolot27 tolot27 approved these changes

@lenilsas lenilsas lenilsas approved these changes

Assignees
No one assigned
Labels
blocked Depends on another feature or request fix This will fix a bug
Projects
None yet
Milestone
No milestone
Development

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