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

EVENTS: Mailtext Bestätigungsmail abhängig von Konfiguration Kurs #80

Closed
10 of 11 tasks
chrusu opened this issue Feb 23, 2022 · 9 comments
Closed
10 of 11 tasks

EVENTS: Mailtext Bestätigungsmail abhängig von Konfiguration Kurs #80

chrusu opened this issue Feb 23, 2022 · 9 comments

Comments

@chrusu
Copy link

chrusu commented Feb 23, 2022

Die Kommunikation muss noch verbessert werden. Aktuell ist aus dem bestätigungs Mail mit dem PDF nicht ersichtlich, dass die Anmeldung noch nicht final ist.

Tech spec

  • Umsetzung im Core

TODO

  • In Event::ParticipationsController#set_success_notice einen weiteren Spezialfall einbauen, der eine andere Message ausgibt falls entry.application == true ist
  • In banner_participating eine Unterscheidung mit einer anderen Übersetzung einführen, falls @user_participation.application == true ist
  • Neues Mail welches statt Event::ParticipationMailer.confirmation ausgelöst wird falls participation.application == true ist
    • Neuen Mailtext als CustomContent in https://github.com/hitobito/hitobito/blob/master/db/seeds/custom_contents.rb#L329 erfassen
    • Im Event::ParticipationMailer eine neue Methode für das neue Mail erstellen (viel Spass bei der Namensfindung, Warteliste und provisorische Anmeldung sind bereits von anderen Konzepten in hitobito besetzt)
    • Im Event::ParticipationConfirmationJob das neue Mail auslösen, falls participation.application == true ist
  • PDF-Export anpassen gemäss Anforderungen (genauere Beschreibung vom Cevi ist noch offen)
  • Specs schreiben
  • Kunde wegen Übersetzungen und neu verfügbarem CustomContent informieren
  • Mit angemessener Rolle "durchklicken"
  • CHANGELOG-Eintrag unter "unreleased" unten hinzufügen
@nchiapol
Copy link
Collaborator

nchiapol commented Apr 2, 2022

Aus den für den User sichtbaren Infos ist nicht ersichtlich, dass seine Anmeldung bei einem Kurs erst auf dem "Anmelden"-Tab gelandet ist und er noch manuell dem Kurs zugewiesen werden muss.

  • Bestätigung nach Anmeldung
    image

  • Kurs-Seite nach Anmeldung
    image

  • Mail-Text "Anmelde-Bestätigung"

  • Inhalt Anmelde-PDF

Die Texte können zum Teil via Einstellungen angepasst werden. Da die Texte aber für alle Anlässe gelten und der Prozess zwischen Kursen und anderen Anlässen unterschiedlich ist, lässt sich das nicht darüber lösen.

Bei Kursen (falls #79 umgesetzt wird nur solche mit Ablauf (b)) soll an all diesen Stellen klar ersichtlich sein, dass die Anmeldung erfasst wurde, aber noch manuell geprüft wird, bevor sie definitiv ist.

Für die Texte in Einstellungen bedeutet dies wohl zwei verschiedene Mail-Vorlagen.

(Ab dem Zeitpunkt der Zuweisung sollten dann auf der Kurs-Seite wieder die definitive Bestätigung angezeigt werden :-)

@mtnstar
Copy link
Contributor

mtnstar commented Nov 3, 2022

@nchiapol ich habe den Betreff des neuen Mails mal 'Anfrage Teilnahme' bezeichnet damit gleich klar ist das die Anmeldung in dem beschriebenen Fall noch nicht bestätigt ist.

ich frage mich gerade ob es Sinn macht das bei diesem Mail auch ein PDF als Anhang mitgeschickt wird. Im PDF für die definitive Anmeldung sind ja schon recht detaillierte Informationen sowie auch Felder für Unterschrift usw. vorhanden.

Ist es aus deiner Sicht ok wenn wir das PDF für die Anfrage weglassen oder willst du uns noch den Inhalt dazu geben?

PDF für die definitive Anmeldung sieht so aus:

image

@nchiapol
Copy link
Collaborator

nchiapol commented Nov 3, 2022

Vielen Dank für die Rückfrage und den Hinweis auf das PDF.

  • Bezeichnung: eine klare Bezeichnung macht sicher Sinn. "Anfrage Teilnahme" scheint mir aber fast etwas schwach. Alternativen wären "Voranmeldung erhalten", "provisorische Anmeldung". Ich frage mal in unserem Team noch nach ob jemand eine besonders gute Ideen für den Betreff hat.
  • PDF: Das ist ein guter Hinweis. Dieses PDF muss mit dem "Anfrage Teilnahme" mail versendet werden. Ein Verwendungszweck des zweistufigen Ablaufs ist, dass wir so die Unterschrift kontrollieren können.

@carlobeltrame
Copy link
Member

Hinweis: Die Bezeichnungen "Warteliste" und "provisorische Anmeldung" sind bereits von anderen Konzepten in hitobito besetzt.

@mtnstar
Copy link
Contributor

mtnstar commented Nov 6, 2022

@nchiapol super, dann verwenden wir doch den Begriff Voranmeldung.

soll dann der Inhalt des PDFs gleich sein wie das von der Teilnahmebestätigung? Ausser der Titel heisst anstatt Kursanmeldung -> Voranmeldung ?

kannst du mir noch Bescheid geben falls es im PDF selber andere Inhalte benötigt?

@nchiapol
Copy link
Collaborator

nchiapol commented Nov 7, 2022

  • Begriff: "Voranmeldung" passt.
  • PDF: Der Inhalt des PDFs kann aus meiner Sicht identisch sein und auch der Titel "Kursanmeldung" ist ok. Das zweite Mail nach der Zuweisung zum Kurs braucht dann kein PDF mehr. (Macht aber auch nichts, wenn dort das PDF noch einmal verschickt wird.)

Der Ablauf ist also:

  1. TN meldet sich an
  2. TN erhält "Voranmeldung erhalten" E-Mail mit PDF der Anmeldung. Im Mail-Text erläutern wir, dass die Anmeldung erst definitiv wird, wenn wir alle Bedingungen kontrolliert und falls nötig die relevanten Unterschriften erhalten haben.
  3. Wir kontrollieren die Bedingungen/TN sendet das unterschrieben Dokument ein.
  4. Wir verschieben TN auf dem "Anmeldungen"-Tab nach "Zugeteilte Teilnehmende"
  5. TN erhält zweites Mail mit Info, dass Anmeldung bestätigt wurde und nun definitiv gültig ist.

(Eigentlich könnte man sagen, dass nicht das Mail unter (2) sondern jenes unter (5) neu ist.)

@nchiapol
Copy link
Collaborator

Habe das eben angeschaut, sieht sehr hübsch aus. Danke.

Offene Anliegen schreibe ich in #79, da die verbleibenden Wünsche primär die Effekte der verschiedenen Konfigurationen betrifft.

@richardjubla
Copy link

Lieber @ThomasEllenberger

Wie kann ich nachvollziehen, was im Zusammenhang mit diesem Change (Referenz aus Changelog Version 1.28) verändert wurde?

@richardjubla
Copy link

Bezeichnend für die Situation, es betrifft die Kursverwaltung....

  • Dieser Change wird/soll gemäss beschrieb im Core umgesetzt
  • Der Change ist mit Version 1.3 im Changelog von 1.28 bei uns (jubla) neu (rückwirkend?) aufgetaucht
  • Gemäss Informationen gehen wir davon aus, dass die Anliegen zusammen mit SportDB Export bei Lager: NDBJS_PERS_NR leer cevi/hitobito_cevi#79 gemacht wurden und in den PR's von diesem Issue umgesetzt wurden.
    - Uns ist nicht klar, was und wie nun auf core/jubla für uns relevant oder zu testen ist

Version 1.28.30.2
image
Version 1.30.20
image

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

No branches or pull requests

6 participants