nextcloud / mail

💌 Mail app for Nextcloud
https://apps.nextcloud.com/apps/mail
GNU Affero General Public License v3.0
846 stars 259 forks source link

Missing warning before sending to many recipients #6461

Open ChristophWurst opened 2 years ago

ChristophWurst commented 2 years ago

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 https://github.com/nextcloud/mail/pull/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 https://github.com/nextcloud/mail/pull/6031

wriver4 commented 2 years ago

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.

miaulalala commented 2 years ago

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

ChristophWurst commented 2 years ago

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

ChristophWurst commented 2 years ago

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

miaulalala commented 2 years ago

Blocked by #7517

miaulalala commented 1 year ago

Blocked by https://github.com/nextcloud/mail/pull/7675

miaulalala commented 8 months ago

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