Skip to content

[stable22] Allow users to choose a different email for notifications - #28780

Merged
skjnldsv merged 8 commits into
stable22from
backport/28422/stable22
Sep 10, 2021
Merged

[stable22] Allow users to choose a different email for notifications#28780
skjnldsv merged 8 commits into
stable22from
backport/28422/stable22

Conversation

@backportbot-nextcloud

@backportbot-nextcloudbackportbot-nextcloudBot commented Sep 10, 2021

Copy link
Copy Markdown

⚠️ This backport had conflicts and is incomplete ⚠️

backport of #28422

@backportbot-nextcloudbackportbot-nextcloudBot added enhancement feature: users and groups pending documentation This pull request needs an associated documentation update labels Sep 10, 2021
@backportbot-nextcloudbackportbot-nextcloudBot added this to the Nextcloud 22.1.2 milestone Sep 10, 2021
- to make it reusable
- needed for local email verification
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
- mails added by (sub)admins are automatically verified
- provisioning_api controller as verification endpoint
- IAccountProperty gets a locallyVerified property
- IPropertyCollection gets a method to fetch an IAccountProperty by value
- an remove equivalent was already present
- AccountManager always initiates mail verification on update if necessary
- add core success template for arbitrary title and message
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@blizzz
blizzzforce-pushed the backport/28422/stable22 branch from bec4e4d to cc9758eCompareSeptember 10, 2021 11:21
@blizzzblizzz added the 3. to review Waiting for reviews label Sep 10, 2021
- this is to avoid automatic confirmation by certain softwares that open
links
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
- specific getters and setters on IUser and implementation
- new notify_email field in provisioning API
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
- there will be times when it is necessary to reset this value for sure
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@skjnldsv
skjnldsv merged commit 317e04d into stable22Sep 10, 2021
@skjnldsv
skjnldsv deleted the backport/28422/stable22 branch September 10, 2021 13:29
@blizzzblizzz mentioned this pull request Sep 23, 2021
4 tasks
@MichaIngMichaIng modified the milestones: Nextcloud 22.1.2, Nextcloud 22.2.0Sep 24, 2021
@nickvergessennickvergessen removed the pending documentation This pull request needs an associated documentation update label Nov 8, 2022
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@blizzz@skjnldsv@nickvergessen@MichaIng