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

Feature/sac 249 signup wizard subsidy #2567

Closed
wants to merge 2 commits into from

Conversation

amaierhofer
Copy link
Contributor

No description provided.

@amaierhofer amaierhofer force-pushed the feature/sac-249-signup-wizard-subsidy branch from 7022ec3 to b85cb71 Compare April 15, 2024 08:33
@amaierhofer amaierhofer force-pushed the feature/sac-249-signup-wizard-subsidy branch from b85cb71 to bbe1244 Compare April 15, 2024 08:34
@amaierhofer amaierhofer requested a review from codez April 15, 2024 08:55
@kronn kronn deleted the branch master April 15, 2024 13:42
@kronn kronn closed this Apr 15, 2024
@@ -69,7 +69,9 @@ def create
send_notification_email
end
end
respond_with(entry, success: created, location: return_path)

location = after_create_path if entry.valid?
Copy link
Contributor

Choose a reason for hiding this comment

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

Sollte das nicht if created prüfen?

@daniel-illi daniel-illi reopened this Apr 16, 2024
@daniel-illi daniel-illi changed the base branch from sac-master to master April 16, 2024 08:38
Copy link
Contributor

@codez codez left a comment

Choose a reason for hiding this comment

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

Da fehlte in den Tickets der Blick übers Ganze. Die Anmeldung via Wizard soll erst erstellt werden, wenn der letzte Schritt abgeschlossen ist. Die Daten müssen bis dann (wohl im Browser) nur zwischengespeichert werden. Der erste Schritt mit den Personendaten kann falls einfacher davon ausgenommen werden, da dies ja noch keine Participation erzeugt. Die Daten der weiteren Schritte Zusatzdaten, Subventionen und Zusammenfassung jedoch sollen erst beim letzten Submit persistiert werden. Entsprechend sind wohl diese Anpassungen hier in der Form nicht zielführend. Ansonsten müssten wir uns mit nicht abgeschlossenen Anmeldungen herumschlagen...

@codez
Copy link
Contributor

codez commented Apr 19, 2024

Gemerged mit #2569

@codez codez closed this Apr 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

EVENTS: Anmeldewizard - Neuer Schritt Subventionsbeitrag
4 participants