Skip to content

Reworded the event text to be less vague#1726

Merged
MImran2002 merged 8 commits into
developmentfrom
fritzj2-zhytkovd-1358
Jun 17, 2026
Merged

Reworded the event text to be less vague#1726
MImran2002 merged 8 commits into
developmentfrom
fritzj2-zhytkovd-1358

Conversation

@zhytkovd

@zhytkovd zhytkovd commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Issue Description

Fixes #1358

  • The original text, "This event is engagement or education" was vague and confusing for some users. This was changed to be clearer.

Changes

  • The text now reads "This event is for education or engagement only"
image

Testing

  • The checkbox still works.
  • The text appears on all of the Celts programs' menus.
  • The text persists after the event is created.

Copilot AI review requested due to automatic review settings June 16, 2026 20:50

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the Create/Edit Event UI copy to make the “engagement/education” event-type checkbox label clearer for end users (fixes #1358).

Changes:

  • Replaces the vague label text “This event is engagement or education.” with “This event is centered around participation, learning, or both.”

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@fritzj2 fritzj2 requested a review from brightfietsop-ux June 17, 2026 13:13
@brightfietsop-ux

Copy link
Copy Markdown

I reviewed your work, and I approve of it.

@MImran2002 MImran2002 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Talked to Brian the wording seems a bit confusing still so he preferred this event is for education or participation only.

Copilot AI review requested due to automatic review settings June 17, 2026 18:53

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review is ineligible. To be eligible to request a review, you need a paid Copilot license, or your organization must enable Copilot code review.

@fritzj2

fritzj2 commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Made the requested changes to the text as listed below.

Talked to Brian the wording seems a bit confusing still so he preferred this event is for education or participation only.

Comment thread app/templates/events/createEvent.html Outdated
</div>
<div class="form-check form-switch">
<label class="custom-control-label" for="checkEngagement"><strong>This event is engagement or education.</strong></label>
<label class="custom-control-label" for="checkEngagement"><strong>This event is for education or engagement only.</strong></label>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

should be participation not engagement

Copilot AI review requested due to automatic review settings June 17, 2026 19:52

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review is ineligible. To be eligible to request a review, you need a paid Copilot license, or your organization must enable Copilot code review.

@MImran2002 MImran2002 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

looks good

@github-actions

Copy link
Copy Markdown

View Code Coverage

@MImran2002 MImran2002 merged commit a2d4c01 into development Jun 17, 2026
5 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.

Create Event Grammar Issue

5 participants