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: Selbsteinschreibung zu Zusatzsektion funktioniert nicht #235

Closed
carlobeltrame opened this issue Jan 19, 2024 · 3 comments · Fixed by #264
Closed

PEOPLE: Selbsteinschreibung zu Zusatzsektion funktioniert nicht #235

carlobeltrame opened this issue Jan 19, 2024 · 3 comments · Fixed by #264
Labels
bug Something isn't working interest-sac

Comments

@carlobeltrame
Copy link
Member

Aktuelles Verhalten

Wenn man versucht einer Zusatzsektion beizutreten, kommt nach dem Bestätigungs-Dialog nur eine weisse Seite.

Erwartetes Verhalten

Einschreibung bei der Neuanmeldungs-Gruppe für eine Zusatzsektion funktioniert normal.

Schritte zum Reproduzieren

  1. Den Selbstregistrierungslink einer Sektion A heraussuchen
  2. Ein Mitglied einer anderen Sektion B imitieren
  3. Den Selbstregistrierungslink von Sektion A besuchen
  4. "Eintritt als Zusatzsektion" aktivieren
  5. Beitreten anklicken
  6. Im Dialog "Beitrag beantragen" klicken
  7. Weisser Screen taucht auf

Weitere Infos, Kontext

Mit einer lokal geseedeten Person kommt man schon gar nicht so weit: Da gibts schon Validierungsfehler von irgendeiner simulierten Rolle nach Schritt 3.
Wenn möglich, dies gleich mitfixen, z.B. via Anpassung der Seeds. Es sieht so aus als fehlen auf irgendeiner geseedeten Rolle irgendwelche Timestamps, aber ich kann nicht genau erkennen ob man einfach diese Spalten befüllen müsste, oder ob die Logik in mitglied_no_overlap_validation.rb mehr Fehler-Handling braucht.

@carlobeltrame carlobeltrame added bug Something isn't working interest-sac labels Jan 19, 2024
@daniel-illi daniel-illi self-assigned this Jan 23, 2024
@daniel-illi
Copy link
Contributor

Es wird versucht, eine Rolle vom falschen Typ zu erstellen, eine Group::SektionsNeuanmeldungenNv::Neuanmeldung statt eine Group::SektionsNeuanmeldungenNv::NeuanmeldungZusatzsektion

@daniel-illi
Copy link
Contributor

@carlobeltrame die Selbsteinschreibungs-Funktionalität scheint mir noch nicht fertig implementiert zu sein.
Im Form gibt es zwar ein "Eintritt als Zusatzsektion", dieses hat aber keine Wirkung. Es wird immer eine Rolle erstellt von dem Typ, der auf der Gruppe konfiguriert ist für die Selbstregistrierung.
IMO ist für die Selbsteinschreibung die Selbstregistrierungsrolle irrelevant. Der Rollentyp müsste anhand der gewählten Option eruiert werden.
Ausserdem sollten abhängig von den bestehenden Rollen der Person nur gültige Optionen zur Verfügung stehen.

@mtnstar
Copy link
Member

mtnstar commented Jan 25, 2024

@carlobeltrame ich denke bei der Selbsteinschreibung sind auch noch nicht alle möglichen Funktionen implementiert.
wir können uns gerne mal über alle mögliche Use Cases da unterhalten

daniel-illi added a commit that referenced this issue Jan 29, 2024
daniel-illi added a commit that referenced this issue Jan 29, 2024
daniel-illi added a commit that referenced this issue Jan 29, 2024
daniel-illi added a commit that referenced this issue Jan 29, 2024
@daniel-illi daniel-illi removed their assignment Jan 29, 2024
@mtnstar mtnstar self-assigned this Jan 30, 2024
mtnstar pushed a commit that referenced this issue Jan 30, 2024
mtnstar added a commit that referenced this issue Jan 30, 2024
@mtnstar mtnstar removed their assignment Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working interest-sac
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants