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

Missing warning before sending to many recipients #6461

Open
ChristophWurst opened this issue May 16, 2022 · 7 comments
Open

Missing warning before sending to many recipients #6461

ChristophWurst opened this issue May 16, 2022 · 7 comments

Comments

@ChristophWurst
Copy link
Member

ChristophWurst commented May 16, 2022

Steps to reproduce

  1. Send a message to 11 people

Expected behavior

The app warns that the message will go to lots of recipients thanks to #4838.

Actual behavior

The app just sends without confirmation.

Mail app version

v1.12.0

Mailserver or service

No response

Operating system

No response

PHP engine version

No response

Web server

No response

Database

No response

Additional info

Regression of #6031

@wriver4
Copy link

wriver4 commented May 16, 2022

In the discussion of email warning for too many, you mentioned 11. I was sending an email using 12 people in the TO: field, and the first one on the mail list did not get sent and errored out. It was stuck in the outbox. Sent email to the same account, and they were successful, did not get stuck in the outbox.

@ChristophWurst ChristophWurst moved this to 📄 To do (5-20 entries) in 💌 📅 👥 Groupware team May 17, 2022
@miaulalala miaulalala self-assigned this Jun 13, 2022
@miaulalala
Copy link
Contributor

Since the send action is divorced from the message creation, at which point should the error be displayed?

@ChristophWurst
Copy link
Member Author

For better UX I'd say when the message is added to the outbox

@ChristophWurst
Copy link
Member Author

Label dropped because it was a duplicate of the attribute at https://github.com/orgs/nextcloud/projects/61. The priority does not change.

@miaulalala

This comment was marked as outdated.

@miaulalala

This comment was marked as outdated.

@miaulalala
Copy link
Contributor

On converting a draft to an Outbox message the converting should fail if there's too many recipients and allow a force send option.

@miaulalala miaulalala moved this from 🏗️ In progress to 📄 To do in 💌 📅 👥 Groupware team Jul 23, 2024
@jancborchardt jancborchardt moved this to 🏗️ At engineering in 🖍 Design team Oct 16, 2024
@ChristophWurst ChristophWurst assigned kesselb and unassigned miaulalala Nov 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 📄 To do
Status: 🏗️ At engineering
Development

Successfully merging a pull request may close this issue.

4 participants