Skip to content

Add required indicators and validation for event fields and custom questions#2135

Merged
trista-chen-29 merged 2 commits into
devfrom
trista/event_form_required_fields
May 3, 2026
Merged

Add required indicators and validation for event fields and custom questions#2135
trista-chen-29 merged 2 commits into
devfrom
trista/event_form_required_fields

Conversation

@trista-chen-29

@trista-chen-29 trista-chen-29 commented May 3, 2026

Copy link
Copy Markdown
Collaborator

Changes

  • Add * indicators for required event fields: name, date, time, location
  • Show * for minimum visible role only when visibility is private
  • Add required indicators for custom registration question text
  • Validate blank custom question text and choice options before creating/editing events

Screenshots

Screenshot 2026-05-02 at 8 55 32 PM Screenshot 2026-05-02 at 8 55 39 PM Screenshot 2026-05-02 at 10 29 43 PM

@steeevin88 steeevin88 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

thanks for adding this lol, i noticed this in the past but totally forgot about it

@steeevin88 steeevin88 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

on second thought, does this mean the text for required CUSTOM questions won't work? if not can we add the fix in this PR too?

@trista-chen-29 trista-chen-29 force-pushed the trista/event_form_required_fields branch from a890f0d to be81f62 Compare May 3, 2026 05:32
@trista-chen-29 trista-chen-29 changed the title Fix required field indicators in event form Add required indicators and validation for event fields and custom questions May 3, 2026

@KhoaNguyen706 KhoaNguyen706 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LWTM

@trista-chen-29 trista-chen-29 merged commit fb1bae3 into dev May 3, 2026
4 checks passed
@trista-chen-29 trista-chen-29 deleted the trista/event_form_required_fields branch May 3, 2026 06:21
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.

3 participants