fix: add resend speakers checkbox on summit speakers email send popup - #1064
fix: add resend speakers checkbox on summit speakers email send popup#1064tomrndom wants to merge 3 commits into
Conversation
Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Essentials Run ID: 📒 Files selected for processing (4)
Included review availability: 3 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour. 📝 WalkthroughWalkthroughThe speaker email workflow adds a resend-speakers checkbox, passes its value through the page and action layers, and includes it as ChangesSpeaker email resend
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to The email confirmation dialog now lets administrators resend messages to already-sent speakers or submitters, and the selected setting is sent with each email request. No current merge-blocking risk remains. Sequence Diagram(s)sequenceDiagram
participant SpeakerEmailModal
participant SummitSpeakersPage
participant sendSpeakerEmails
participant EmailRequest
SpeakerEmailModal->>SummitSpeakersPage: select resend speakers
SummitSpeakersPage->>sendSpeakerEmails: submit shouldResendSpeakers
sendSpeakerEmails->>EmailRequest: send should_resend
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>
Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>
ref: https://app.clickup.com/t/9014802374/86bbtnft1
Signed-off-by: Tomás Castillo tcastilloboireau@gmail.com
Summary by CodeRabbit