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

Sepa Check temporär deaktivieren #585

Merged
merged 7 commits into from
Jan 22, 2025

Conversation

JohannMaierhofer
Copy link

Ich habe jetzt das temporäre deaktivieren des SEPA Check entsprechend der Diskussion in #581 implementiert.

  • Im Abrechnung View gibt es den Schalter für das Deaktivieren
    Bildschirmfoto_20250117_191437
  • Will man die Checkbox setzen kommt ein Dialog der den User warnt
    Bildschirmfoto_20250117_191505
  • Ist das Flag gesetzt wird der SEPA Check beim Abrechnungslauf übersprungen
  • Einen Fehler den der Check zurück liefert wird wie bisher im Monitor Log ausgegeben
  • Dann habe ich den Check in das Mitglied verschoben so dass der gleiche Code vom Abrechnungslauf und manuellem Erzeugen benutzt werden kann. Auch den ConfirmDialog habe ich nur einmal übrig gelassen
  • Beim manuellen Erzeugen einer Lastschrift wird jetzt gleich der Check beim Mitglied aufgerufen
  • Geht der Check schief wird der Fehler der zurückgegeben wird in einem Dialog ausgegeben
    Bildschirmfoto_20250117_191757
  • Hier kann der User entscheiden ob er abbricht oder weiter macht

@JohannMaierhofer JohannMaierhofer changed the title Update sepa check Sepa Check temporär deaktivieren Jan 17, 2025
@lenilsas
Copy link

Können wir bitte erst #547 übernehmen bevor wir weitere Änderungen an der Abrechnung machen, ich habe hier so viele Änderungen gemacht, dass es viele mergekonflikt gibt.

@JohannMaierhofer
Copy link
Author

Können wir bitte erst #547 übernehmen bevor wir weitere Änderungen an der Abrechnung machen, ich habe hier so viele Änderungen gemacht, dass es viele mergekonflikt gibt.

Ja, können wir machen.

@JohannMaierhofer
Copy link
Author

Merge Konflikt behoben

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.

Wir sollten den AbrechnungsView auch noch überarbeiten, langsam werden es sehr viele Wahlmöglichkeiten, da wären zwei Spalten besser. Können wir aber auch in einem extra PR mit #563 machen.

@JohannMaierhofer
Copy link
Author

Wir sollten den AbrechnungsView auch noch überarbeiten, langsam werden es sehr viele Wahlmöglichkeiten, da wären zwei Spalten besser. Können wir aber auch in einem extra PR mit #563 machen.

Ja, mit den extra issue machen.

lenilsas
lenilsas previously approved these changes Jan 21, 2025
@tolot27 tolot27 merged commit 98642ff into openjverein:master Jan 22, 2025
2 checks passed
@JohannMaierhofer JohannMaierhofer deleted the update_sepa_check branch January 22, 2025 18:46
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

Successfully merging this pull request may close these issues.

3 participants