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

fix: Reset notification email when deleting additonal email #45487

Merged
merged 2 commits into from
Aug 2, 2024

Conversation

Pytal
Copy link
Member

@Pytal Pytal commented May 23, 2024

Summary

When deleting an additional email that is set as the notification email, the deleted email would remain as the notification email. We fix this by automatically resetting the notification email.

Before After
image image

Revives #36812

Checklist

@Pytal Pytal added this to the Nextcloud 30 milestone May 23, 2024
@Pytal Pytal requested review from blizzz and artonge May 23, 2024 18:24
@Pytal Pytal self-assigned this May 23, 2024
@Pytal Pytal requested a review from susnux May 23, 2024 18:25
This was referenced Jul 30, 2024
@blizzz
Copy link
Member

blizzz commented Aug 2, 2024

/compile amend /

@susnux susnux force-pushed the fix/reset-notification-email branch from 089f175 to 7a4c533 Compare August 2, 2024 10:38
@susnux susnux enabled auto-merge August 2, 2024 10:38
Signed-off-by: Christopher Ng <[email protected]>
@susnux susnux force-pushed the fix/reset-notification-email branch from 7a4c533 to 427c27a Compare August 2, 2024 10:42
@susnux susnux merged commit 1a51afd into master Aug 2, 2024
169 checks passed
@susnux susnux deleted the fix/reset-notification-email branch August 2, 2024 13:11
@Altahrim Altahrim mentioned this pull request Aug 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants