Skip to content
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

Closed
marclaporte opened this issue Nov 8, 2021 · 7 comments
Closed

Soft and/or hard limit to number of recipients #531

marclaporte opened this issue Nov 8, 2021 · 7 comments

Comments

@marclaporte
Copy link
Member

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.

@dumblob
Copy link
Member

dumblob commented Nov 8, 2021

We should have warnings to help users.

Yep, persistent (i.e. visible during the whole compose session - even if opened from a draft folder) warning. But please no hard limit.

@jasonmunro
Copy link
Member

@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.

@marclaporte
Copy link
Member Author

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.

@jasonmunro
Copy link
Member

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

@marclaporte
Copy link
Member Author

Your proposal works for me :-)

@jasonmunro
Copy link
Member

committed so closing this.

@marclaporte
Copy link
Member Author

For the record: 293d1af

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

No branches or pull requests

3 participants