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

Spendenbescheinigung per Mail versenden, implementiert #191 #218

Merged
merged 12 commits into from May 20, 2024

Conversation

JohannMaierhofer
Copy link

Der Request implementiert das Feature #191
Es gibt einen neuen Menüpunkt "Spendenbescheinigungen versenden" bei Spendenbescheinigungen, multi Selection ist aktiv:
Screenshot_20240426_090846
Dann kommt ein Dialog wo man entscheiden kann ob die Spendenbescheinigungen gedruckt werden sollen. Sind sie schon gedruckt kann man ohne neues drucken weiter gehen.
Screenshot_20240418_180941
Der View zum senden schaut dann so aus. Im Info Feld liste ich Mitglieder auf die keine Mailadresse haben. Auch wird aufgelistet ob für eine Spendenbescheinigung kein Mitglied existiert. Das ist der Fall wenn man die Spendenbescheinigung im Spendenbescheinigung Liste View manuell erzeugt. Für die muss man dann halt die Bescheinigungen per Brief versenden.

Screenshot_20240426_091226

Über einen Schalter in den Einstellungen lässt sich auswählen ob der Mailanhang (Spendenbescheinigung) mit der Mail in der Datenbank gespeichert werden soll.

Screenshot_20240426_091830

dippeal
dippeal previously approved these changes May 18, 2024
Copy link
Member

@dippeal dippeal left a comment

Choose a reason for hiding this comment

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

Schöne Implementierung. Meine Tests mit Standard Formular und individuallem Formular gingen problemlos. Auch die Erkennung "Kein Mitgliedskonto" hat funktioniert.

@SchachtnerTh
Copy link

Wir haben einige Vorlagen erstellt, um die Mails ansprechend mit HTML zu formatieren.
Wenn ich im Kontextmenü bei "Spendenbescheinigungen" den Eintrag "E-Mail an Spender" auswähle, kann ich eine solche Vorlage auswählen. Wenn ich "Spendenbescheinigungen versenden" auswähle, habe ich diese Möglichkeit nicht. Wäre es denn möglich, das hier auch einzubauen?
Dann wäre das Verhalten beim Versenden von Mails in der Anwendung auch konsistenter.
(Hat aber nix mit dem Review zu tun, ist nur eine Anmerkung...)

@SchachtnerTh
Copy link

Beim Versenden von Mails mit kurzem Text passt die Anzeige:
grafik

Beim Versand von Mails mit einem längeren Text, sieht das vor dem Versand auch noch gut aus:
grafik

Nach dem Versand ist aber die Info-Box abgeschnitten und man kann die Informationen darin nicht mehr lesen:
grafik

@SchachtnerTh
Copy link

Mir fällt jetzt noch ein Feature ein, das in diesem Zusammenhang vielleicht ganz cool wäre:
Man bekommt ja die Namen der Leute angezeigt, für die die Spendenbescheinigung nicht per Mail verschickt werden kann. Die muss ich dann ausdrucken.
Ich kann mir die Leute aber vielleicht nicht alle merken. Es wäre doch cool, wenn es eine Funktion geben würde, mit der ich die Spendenbescheinigungen, die nicht per Mail versandt werden können, ausdrucken kann.
Vielleicht würde es ja reichen, einen Button beim E-Mail-Versand-Dialog zu haben, der alle Bescheinigungen markiert, die nicht per Mail verschickt werden können. Die könnte ich dann über das Kontextmenü ganz unkompliziert ausdrucken.

Copy link

@SchachtnerTh SchachtnerTh left a comment

Choose a reason for hiding this comment

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

Super Feature!
Super umgesetzt, finde ich.

Ich hab nur gesehen, dass eine Box ziemlich klein wird.
Wenn Du möchtest, kannst Du das noch ändern.

Außerdem verwende ich für meine Mails Vorlagen. Ich denke, ich würde auch eine oder zwei Vorlagen für die Spendenquittungen machen, wenn das ginge. Wenns nicht so einfach umsetzbar ist, kann ich auch darauf verzichten, aber ich wollt das halt mal anmerken.

Ich würde das aber auch so approven. Gib bitte einfach kurz Bescheid, ob Du das noch ändern möchtest oder nicht, dann approve ich.

@JohannMaierhofer
Copy link
Author

Das mit dem zu kleinen Info Fenster möchte ich schon korrigieren. Das ist wirklich nicht brauchbar.
Das mit den Vorlagen könnte ich mir evtl. in Zukunft nochmal extra anschauen. Es ist jetzt halt so wie es auch bei den Mahnungen und Rechnungen ist. Da wird auch nur ein Text angeboten und bleibt gespeichert.

@JohannMaierhofer
Copy link
Author

Ich habe jetzt den Info Text in ein InfoPanel abgeändert. Da bleibt das Feld sichtbar.
Auch habe ich den Text in den Einstellungen geändert.
Screenshot_20240518_190827

Die Anregung um herauszufinden wer keine Mailadresse hat, um zu wissen, wem man die Bescheinigung per Post schicken muss, habe ich auch schon erkannt. Dazu habe ich einige Filter im Spendenbescheinigung View eingebaut. Da kann man angeben, dass man alle Bescheinigungen ohne Mailadresse sehen will.
Dann kann man hier alle markieren und drucken.
Screenshot_20240518_185448

Die Implementierung basiert auf dem Feature #221. Wenn das übernommen wurde, werde ich das übergeben.
PS: Dann kommen auch Filter bei den Mails und ....

@JohannMaierhofer
Copy link
Author

JohannMaierhofer commented May 18, 2024

Jetzt muss ich wohl noch schauen was passiert wenn das Infopanel zu gross wird. Wenn das nicht scrollt ist das wohl auch nicht die richtige Lösung.

@JohannMaierhofer
Copy link
Author

JohannMaierhofer commented May 18, 2024

Ich glaube jetzt sollte es funktionieren. Warum weiß auch auch nicht so recht, aber die Höhe 300 beim Text hat wohl geholfen.
Auch bei langem Text bleibt das Info Feld sichtbar und man kann in beiden scrollen.
Screenshot_20240518_220956

@JohannMaierhofer
Copy link
Author

Die Möglichkeit Templates auszuwählen ist sicher nützlich.
Ich denke, dass ich in Zukunft einen Button im View einfügen könnte mit dem man dann Templates laden kann. Man würde dann standardmäßig den letzten gesetzten Text bekommen so wie jetzt auch, könnte dann aber auch ein anderes nachladen. Dann wird mann nicht jedesmal extra gefragt ob man ein Template benutzen will, da man doch meistens den gleichen Text verwendet.
Das sollte dann analog im Mahnung und Rechnung View passieren.

@dippeal dippeal merged commit 0c52346 into openjverein:master May 20, 2024
@JohannMaierhofer JohannMaierhofer deleted the feature_mailversenden branch May 20, 2024 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants