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
Add a config to disable e-mail notifications on password/email change through API #14281
Conversation
@simivar can you add a test for this behavior in https://github.com/matomo-org/matomo/blob/3.x-dev/plugins/UsersManager/tests/Integration/APITest.php ? |
@diosmosis done in 7a5ece1 |
@diosmosis I was thinking... wouldn't it be better to move in .ini file |
@simivar the issue is already assigned to @katebutler and I think some work had already been done there. @katebutler did you push some code for #14267 already? @simivar be good to check if an issue is maybe already assigned to someone which means they are working on it already. Re the name: It's not too important and actually the current implementation would also disable it for the UI I think. |
Cheers for this @simivar |
Adds a new config option
api_update_users_email_notifications
that when set to 0 disables all emails fromUserManager::updateUser
API endpoint. #14267