Skip to content

Allow users to choose a different email for notifications #28422

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.

Sign up for GitHub

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

Merged
skjnldsv merged 8 commits into from
Sep 10, 2021
Merged

Allow users to choose a different email for notifications #28422

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
19cc757
move verification token logic out of lost password controller
blizzz Aug 13, 2021
aacaad2
implement verification for additional mails
blizzz Aug 20, 2021
37f510c
fix parameter type hint in phpdoc
blizzz Aug 24, 2021
a20de15
add a job to clean up expired verification tokens
blizzz Aug 25, 2021
0dee717
Confirm mails only per POST
blizzz Aug 27, 2021
4461b9e
enable the user to set a primary (notification) email address (backend)
blizzz Sep 1, 2021
27865d0
use specific email getter where necessary
blizzz Sep 1, 2021
763136a
also allow admins to set the primary email
blizzz Sep 1, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
also allow admins to set the primary email 
- there will be times when it is necessary to reset this value for sure

Signed-off-by: Arthur Schiwon <[email protected]>
  • Loading branch information
@blizzz
blizzz committed Sep 9, 2021
commit 763136ab48cc54fda27fa137392020fda900e114
1 change: 1 addition & 0 deletions apps/provisioning_api/lib/Controller/UsersController.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -777,6 +777,7 @@ public function editUser(string $userId, string $key, string $value): DataRespon
$permittedFields[] = IAccountManager::PROPERTY_WEBSITE;
$permittedFields[] = IAccountManager::PROPERTY_TWITTER;
$permittedFields[] = self::USER_FIELD_QUOTA;
$permittedFields[] = self::USER_FIELD_NOTIFICATION_EMAIL;
} else {
// No rights
throw new OCSException('', OCSController::RESPOND_NOT_FOUND);
Expand Down