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 1 commit into
Conversation
Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>
|
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: 4 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 speaker email popup now lets users resend to already-contacted speakers and forwards that choice with the email request. No current merge-blocking risk is identified. 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 |
ref: https://app.clickup.com/t/9014802374/86bbtnft1
Signed-off-by: Tomás Castillo tcastilloboireau@gmail.com
Summary by CodeRabbit