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

Als SAC Fan die Alpen abonnieren funktioniert nicht #1404

Closed
8 tasks
ThomasEllenberger opened this issue Dec 16, 2024 · 2 comments · Fixed by #1420 or hitobito/hitobito#3091
Closed
8 tasks

Als SAC Fan die Alpen abonnieren funktioniert nicht #1404

ThomasEllenberger opened this issue Dec 16, 2024 · 2 comments · Fixed by #1420 or hitobito/hitobito#3091

Comments

@ThomasEllenberger
Copy link

Ausgangslage

Ich möchte als SAC Fan nun das SAC Die Alpen Abo abonnieren. Als Fan habe ich ein aktiven Fankonto mit hinterlegter E-Mail Adresse.

  • Ich gehe auf die Webseite und starte dort das Onboarding für die Alpen und komme auf den Onboarding Prozess
  • Im Schritt 1 gebe ich meine E-Mail Adresse ein
  • Das System stellt fest, dass meine E-Mail Adresse bereits existiert und leitet mich auf die Login Maske
    image
  • Ich gebe das Passwort ein uns sollte nun in auf Schritt 2 Personalien landen.
    ERWARTETES VERHALTEN: Ich gebe das Passwort ein uns sollte nun auf Schritt 2 Personalien landen und die bekannten Personalien sind geladen.
    TATSÄCHLICHES VERHALTEN: Ich lande auf Schitt 1
    image
  • Ich gebe in Schritt 1 die E-Mail Adresse nochmals ein und lande auf der Personalien Maske mit ausgefüllten bekannten Parametern
    image
  • Ich gebe alle fehlenden muss Parameter ein und gehe weiter zu Schritt 3
  • In Schritt 3 klicke ich auf Abo Kostenpflichtig bestellen und nichts passiert.
    image
    image

Anforderungen

  • Hat man bereits ein SAC-Konto und will die Alpen abonieren, landet man nach dem Eingeben von Mail & PW direkt im zweiten Schritt des Anmeldewizards
  • Der Vorgang kann erfolgreich abgeschlossen werden

Tech-Spec

  • sinngebende Zielrichtung
  • besondere/bekannte Bedingungen und Grenzfälle
  • technische Einschränkungen
  • Ein- und Ausgabeformate beschreiben
  • Umsetzungsplan ergänzen oder anpassen

ToDo

  • Migration erstellen
  • Domainklasse erstellen oder anpassen
  • Model/View/Controller anpassen
  • Neues Feature mit FeatureGate deaktivierbar machen
  • Specs schreiben
  • Kunde wegen Übersetzungen informieren
  • Mit angemessener Rolle "durchklicken"
  • CHANGELOG-Eintrag unter "unreleased" unten hinzufügen
@njaeggi
Copy link
Contributor

njaeggi commented Dec 17, 2024

Das Problem war mehr das Benutzen des Abo Wizards bei eingeloggten Usern, mit der Pull Request habe ich mehrere Probleme gelöst:

  • Email Formular wird für eingeloggte Benutzer gar nicht mehr angezeigt
  • Submit für eingeloggte Person funktioniert wieder

Ein paar kleine Anzeigefehler sind mir zudem noch aufgefallen, welche ich hier mal festgehalten habe: #1423

Eventuell muss das genauere Verhalten noch mit dem SAC besprochen werden, gerade das Info Feld "Fragen zur Mitgleidschaft?" anzuzeigen bei eingeloggten Usern oder doch nicht usw.

Zusaätzlich existiert nun wo der 500er gefixed wurde auch noch dieser BUG: #1425

@njaeggi
Copy link
Contributor

njaeggi commented Dec 18, 2024

Die beiden referenzierten Tickets wurden ebenfalls implementiert, ausserdem sollte #1421 auch funktionieren.

Wichtig ist es jede Wizard Art einmal durchzuklicken, um sicherzugehen das sich jeder korrekt verhält, ob eingeloggt oder nicht!

@njaeggi njaeggi assigned amaierhofer and unassigned njaeggi Dec 18, 2024
amaierhofer added a commit that referenced this issue Dec 18, 2024
* Do not include main email step when logged in and build as current_user

* Simplify build_person method

* Add specs for current steps

* Add model spec for saving records

* Add feature spec for all abo wizards

* Adjust views to not display unwanted info

* Move member_or_applied? to wizards

* General cleanup

* Add spec to check if wizard is viewable

* Fix rubocop

* Fix broken specs

* Fix feature specs

* Cleanup specs a bit

---------

Co-authored-by: Andreas Maierhofer <maierhofer@puzzle.ch>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment