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: Use new structured fields in participation_contact_data validations #2660

Closed
4 tasks done
kronn opened this issue Jun 6, 2024 · 1 comment · Fixed by #2682
Closed
4 tasks done

EVENTS: Use new structured fields in participation_contact_data validations #2660

kronn opened this issue Jun 6, 2024 · 1 comment · Fixed by #2682

Comments

@kronn
Copy link
Member

kronn commented Jun 6, 2024

Aktuelles Verhalten

event_participation_contact_data[housenumber] can/should be validated by now. This leads to errors or a state in which sending the form is not possible anymore.

Gewünschtes Verhalten

Validations of Participation Contact Data should work "like before", just with the new fields. The previous field address should not be part the validations anymore.

Schritte zu reproduzieren

  • have an event with participation-contact-data configured
  • apply

Wann ist der Fehler aufgetreten?

2024-06-06

hitobito version

2.1.5-integration

Organisation

Jubla, but most likely everyone

Screenshots, weiterer Kontext

image

contact_data:1 - Failed to load resource: the server responded with a status of 422 (Unprocessable Content)

turbo.es2017-esm.js:941 
Error: POST https://***/groups/296/events/699/participations/contact_data?event_role%5Btype%5D=Event%3A%3ARole%3A%3AParticipant 422

Originally posted by @richardjubla in #2595 (comment)

ToDos

  • Check validations
  • Verify/adapt existing config of events
  • write a migration to change requirement of address to street
  • Adapt specs if needed
@richardjubla
Copy link
Contributor

Vielen Dank! Ich mache hier einen auf "Panik": Im nächsten Release sind die Youth-Verbände voll in den Lageranmeldungen drin. Es wäre sehr ungünstig, wenn die Events dann nicht korrekt funktionieren sollten.

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

Successfully merging a pull request may close this issue.

3 participants