Skip to content

Add reminder emails to Event Management#1433

Open
RJV333 wants to merge 3 commits intomainfrom
938_add_event_remind_email_to_event_management
Open

Add reminder emails to Event Management#1433
RJV333 wants to merge 3 commits intomainfrom
938_add_event_remind_email_to_event_management

Conversation

@RJV333
Copy link
Collaborator

@RJV333 RJV333 commented Mar 15, 2026

Closes [#938 ]

What is the goal of this PR and why is this important?

Updates Event Management with the ability to send Reminder emails. 'Send Reminder' button sends user to /remind page that features a preview of the reminder email, and a selectable table of recipients showing name/email/payment status

Screen.Recording.2026-03-15.at.4.02.47.PM.mov

@RJV333 RJV333 self-assigned this Mar 15, 2026
end
end

def remind
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we call this preview_reminder? The word remind sounds like taking the actual action of reminding. This action deals with the preview.

Copy link
Collaborator

@jmilljr24 jmilljr24 left a comment

Choose a reason for hiding this comment

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

@RJV333 Love this feature! Looks really good.

Could we make the link to the preview page match the style of the other links on the manage event reg page? Also the partial for the new page seems to be missing for this PR.

Image Image

</p>
</div>
<div class="flex gap-2">
<% if allowed_to?(:remind?, @event) %>
Copy link
Collaborator

Choose a reason for hiding this comment

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

This page is admin only so I don't think we need to check a policy for this specific link.

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