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

[FIX]smtp/modules.php: Fix "Message contains more than the maximum number of recipients" on reply #1004

Conversation

IrAlfred
Copy link
Member

@IrAlfred IrAlfred commented May 6, 2024

This pull request fixes the message warning "Message contains more than the maximum number of recipients, proceed with caution" even when we select REPLY instead of REPLY-ALL for a mail that contains more than the maximum number of recipients in To, Cc, or Bcc

Related Task

modules/smtp/modules.php Outdated Show resolved Hide resolved
modules/smtp/modules.php Outdated Show resolved Hide resolved
modules/smtp/modules.php Outdated Show resolved Hide resolved
@IrAlfred
Copy link
Member Author

Thank you @josaphatim for your review. I work on it

@IrAlfred IrAlfred force-pushed the fix-warning-limit-to-number-of-recipients-reply branch from d980756 to 92ffa5a Compare May 26, 2024 11:11
lib/module.php Outdated Show resolved Hide resolved
modules/smtp/modules.php Outdated Show resolved Hide resolved
modules/smtp/modules.php Outdated Show resolved Hide resolved
@IrAlfred IrAlfred force-pushed the fix-warning-limit-to-number-of-recipients-reply branch 2 times, most recently from 1b23a30 to 0282538 Compare May 28, 2024 12:40
@IrAlfred IrAlfred force-pushed the fix-warning-limit-to-number-of-recipients-reply branch from 0282538 to a6a6871 Compare May 28, 2024 13:24
@josaphatim josaphatim merged commit 146715a into cypht-org:master May 28, 2024
6 checks passed
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

Successfully merging this pull request may close these issues.

None yet

2 participants