Conversation
Equal-Vote#1270 asks for a line above the Send button giving admins the first two steps when voters report missing emails, so the answer is on screen at the moment they need it rather than in a support thread. Uses the copy from the issue verbatim, in en.yaml as emails.delivery_note rather than hardcoded in the component. Not done here: the issue also floats expanding this into a helper that explains the individual email error messages from Equal-Vote#713 and linking to it. That needs those messages catalogued first, and is a bigger change than a note above a button.
|
Warning Review limit reached
Next review available in: 59 minutes You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
✅ Deploy Preview for bettervoting ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
1 participant
Description
Closes #1270 — a line above the Send button giving admins the first two steps when voters report a missing email, at the moment they are about to send.
Uses the copy from the issue verbatim:
It lives in
en.yamlasemails.delivery_noterather than hardcoded in the component, so it is translatable and editable by a writer without touching code.Not done here
The issue also floats expanding this into a helper that explains the individual email error messages from #713, linked from this note. That needs those messages catalogued first and is a bigger change than a line above a button — worth its own issue if you want it.
Note on overlap
Touches
SendEmailDialog.tsx, as does #1287 (the stale recipient count). The hunks are far apart — that one is ingetVoterCountand the button'sdisabled, this one adds aTypographyabove the button row — so they should merge cleanly in either order. Happy to rebase whichever lands second.Related Issues
Closes #1270