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

PEOPLE: Selfinscription migrieren auf Wizard #2665

Open
4 tasks
daniel-illi opened this issue Jun 7, 2024 · 0 comments
Open
4 tasks

PEOPLE: Selfinscription migrieren auf Wizard #2665

daniel-illi opened this issue Jun 7, 2024 · 0 comments

Comments

@daniel-illi
Copy link
Contributor

daniel-illi commented Jun 7, 2024

In #2654 wurden die Basisklassen für wizards implementiert und in #2662 wurde die "self-registration" auf das neue Wizard Konstrukt migriert. Die "self-inscription" soll nach dem selben schema funktionieren.

Vor der Umsetzung bitte mit @daniel-illi besprechen.

Tech-Spec

Ein POC existiert auf dem branch poc/wizard-registration und ein dazugehöriger Draft PR https://github.com/hitobito/hitobito/pull/2653/files?w=1
Dieser soll als Vorlage für die Implementation verwendet werden.

  • Wizards::InscribeInGroupWizard
    • einzelner step Wizards::Steps::ConfirmInscription
      • Sicherstellen, dass der user in der Gruppe noch keine Rolle vom gleichen Typ besitzt. Der wizard sollte in diesem Fall gar nicht gestartet werden, also schon im Controller prüfen. Falls der wizard doch gestartet wird, raisen
      • erstellt neue Rolle für user in der Gruppe
    • view partial migrieren app/views/groups/self_inscription/_form.html.haml -> views/wizards/steps/_new_user_form.html.haml
  • Groups::SelfInscriptionController anpassen, dass der neue Wizard verwendet wird
    #entry, #model_class etc siehe POC

ToDo

  • Domainklasse erstellen oder anpassen
  • Model/View/Controller anpassen
  • Specs schreiben
  • Mit angemessener Rolle "durchklicken"
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

5 participants