-
Notifications
You must be signed in to change notification settings - Fork 163
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Soft and/or hard limit to number of recipients #531
Comments
Yep, persistent (i.e. visible during the whole compose session - even if opened from a draft folder) warning. But please no hard limit. |
@marclaporte to clarify, you want to have a warning if you reply-all to a message with > X recipients. That is not what the PR does so I just want to clarify before I comment on it. The PR uses JavaScript to trigger the warning on a keyup event, so it would only trigger if you are actually typing in the recipients. Also the way it is coded the message cannot be translated and it's possible it will miscount as recipient lines are notoriously hard to parse properly. |
You understood correctly. But basically it doesn't matter if it's typed in or reply to all, there should be a soft warning at one point. |
I don't think a warning about typing in 20 recipients makes a lot of sense, but we can add that as well if you think it does. I posted some code to the PR that will warn if you load a draft or reply to a message that has > 20 recipients from the to + cc line. Often larger recipient lists are in the bcc field so I chose not to include that toward the warning |
Your proposal works for me :-) |
committed so closing this. |
For the record: 293d1af |
Every once in a while, I'll receive an email with 50+ recipients. And with more recipients, there is a good chance someone will reply to all.
It is usually an error.
Many clients and email services have limits. Ex.:
We should have warnings to help users.
The text was updated successfully, but these errors were encountered: