nextcloud / nextcloud/mail

Missing warning before sending to many recipients

Open
#6,461 7 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

1. to develop bug feature: outbox regression
Dominant language
JavaScript
Stars
1k
Forks
348
Avg merge
12h 28m
Merged PRs (30d)
91

Description

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

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by comparing the missing behavior with pull request 4838 and the regression noted in pull request 6031. Reproduce sending a message to 11 people, then verify that a confirmation warning appears before sending and that the regression is covered.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
frontend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.