Skip to content

Ensure at least one checkbox is checked in mandatory multiple questions#1

Merged
remi-filament merged 14 commits into16.0-add-website_event_questions_multiplefrom
16.0-mandatory-multiple-questions
Mar 11, 2025
Merged

Ensure at least one checkbox is checked in mandatory multiple questions#1
remi-filament merged 14 commits into16.0-add-website_event_questions_multiplefrom
16.0-mandatory-multiple-questions

Conversation

@Hugo-Trentesaux
Copy link
Member

No description provided.

@Hugo-Trentesaux Hugo-Trentesaux self-assigned this Mar 6, 2025
@remi-filament remi-filament force-pushed the 16.0-add-website_event_questions_multiple branch from 660829c to 54aeed2 Compare March 10, 2025 09:05
@remi-filament remi-filament force-pushed the 16.0-mandatory-multiple-questions branch from 84727c1 to a71efed Compare March 10, 2025 09:06
Copy link
Member

@remi-filament remi-filament left a comment

Choose a reason for hiding this comment

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

Thanks @Hugo-Trentesaux for that work, one comment inline about copied code in JS + you should fix pre-commit (I rebased since at first it failed because of issue with CI deprecated version, but you now have another error coming from your code).

@remi-filament remi-filament merged commit 9ab5256 into 16.0-add-website_event_questions_multiple Mar 11, 2025
4 checks passed
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.

2 participants