Skip to content

Conversation

@nfebe
Copy link
Contributor

@nfebe nfebe commented Jun 4, 2025

This now sets isDefaultEnabled to false. This makes email notifications for file uploads to public links disabled by default (opt-in) for users.

This addresses concerns about new notifications being enabled by default for existing users, leading to unexpected emails.

Related: #46945

@nfebe nfebe requested a review from a team as a code owner June 4, 2025 11:53
@github-project-automation github-project-automation bot moved this to 🏗️ In progress in 📁 Files team Jun 4, 2025
@nfebe nfebe requested review from AndyScherzinger and szaimen June 17, 2025 12:57
This now sets `isDefaultEnabled` to false. This makes email notifications for file uploads
to public links disabled by default (opt-in) for users.

This addresses concerns about new notifications being enabled by
default for existing users, leading to unexpected emails.

Related: #46945

Signed-off-by: nfebe <[email protected]>
@nfebe nfebe force-pushed the fix/public-upload-notification-default branch from cebbbfc to 1f8a230 Compare June 17, 2025 12:57
@nfebe nfebe enabled auto-merge June 17, 2025 12:57
@nfebe nfebe requested review from come-nc and susnux June 17, 2025 13:02
@nfebe nfebe merged commit 137ec92 into master Jun 17, 2025
205 of 212 checks passed
@nfebe nfebe deleted the fix/public-upload-notification-default branch June 17, 2025 14:25
@nfebe
Copy link
Contributor Author

nfebe commented Jun 17, 2025

backport to stable31

@nfebe
Copy link
Contributor Author

nfebe commented Jun 17, 2025

/backport to stable31

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: 🏗️ In progress

Development

Successfully merging this pull request may close these issues.

5 participants