mail Missing warning before sending to many recipients

Steps to reproduce

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

May 16 '22 10:05 ChristophWurst

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.

May 16 '22 15:05 wriver4

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

Sep 23 '22 14:09 miaulalala

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

Sep 23 '22 14:09 ChristophWurst

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

Oct 12 '22 09:10 ChristophWurst

Blocked by #7517

Nov 10 '22 11:11 miaulalala

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

Feb 20 '23 15:02 miaulalala

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

Mar 05 '24 08:03 miaulalala