Skip to content

Do not fail hard on new user mail error#16183

Merged
skjnldsv merged 1 commit into
masterfrom
fix/new-user/mail-error
Jul 2, 2019
Merged

Do not fail hard on new user mail error#16183
skjnldsv merged 1 commit into
masterfrom
fix/new-user/mail-error

Conversation

@skjnldsv

@skjnldsv skjnldsv commented Jul 2, 2019

Copy link
Copy Markdown
Member

Fix #12622

@skjnldsv

skjnldsv commented Jul 2, 2019

Copy link
Copy Markdown
Member Author

/backport to stable16

@skjnldsv

skjnldsv commented Jul 2, 2019

Copy link
Copy Markdown
Member Author

/backport to stable15

Comment thread apps/provisioning_api/lib/Controller/UsersController.php Outdated
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
@skjnldsv
skjnldsv force-pushed the fix/new-user/mail-error branch from bb8d87b to 3340931 Compare July 2, 2019 06:42

@ChristophWurst ChristophWurst left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fine by me :)

@rullzer rullzer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fine by me but please create an issue to extend the response with if the e-mail was send or not

@skjnldsv

skjnldsv commented Jul 2, 2019

Copy link
Copy Markdown
Member Author

Created

@skjnldsv
skjnldsv merged commit 23bcb71 into master Jul 2, 2019
@delete-merged-branch
delete-merged-branch Bot deleted the fix/new-user/mail-error branch July 2, 2019 09:53
@backportbot-nextcloud

Copy link
Copy Markdown

backport to stable16 in #16189

@backportbot-nextcloud

Copy link
Copy Markdown

backport to stable15 in #16190

blizzz added a commit to nextcloud/ldap_write_support that referenced this pull request Jul 17, 2019
* nextcloud/server#16183

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Sign up for free to 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.

Error while creating a new user with an email address

3 participants