Skip to content

Extend reasons for email address - #502

Merged
MorrisJobke merged 1 commit into
masterfrom
bugfix/noid/better-email-reasons
Mar 29, 2021
Merged

Extend reasons for email address#502
MorrisJobke merged 1 commit into
masterfrom
bugfix/noid/better-email-reasons

Conversation

@nickvergessen

Copy link
Copy Markdown
Member

It's a bit confusing to be asked to enter an email address to be able to reset a password while the backend does not allow this at all (LDAP, SAML, ....)

@MorrisJobke

Copy link
Copy Markdown
Member

Could we maybe additionally check for the OCP\User\Backend\ISetPasswordBackend interface on the backend?

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
nickvergessenforce-pushed the bugfix/noid/better-email-reasons branch from 4e8c403 to 83dd668CompareMarch 29, 2021 09:15
@nickvergessen

Copy link
Copy Markdown
MemberAuthor

Added

@MorrisJobke
MorrisJobke merged commit fe51aa8 into masterMar 29, 2021
@MorrisJobke
MorrisJobke deleted the bugfix/noid/better-email-reasons branch March 29, 2021 11:38
@MorrisJobkeMorrisJobke added this to the Nextcloud 22 milestone Mar 29, 2021
@nickvergessen

Copy link
Copy Markdown
MemberAuthor

/backport to stable21

@nickvergessen

Copy link
Copy Markdown
MemberAuthor

/backport to stable20

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.

3 participants

@nickvergessen@MorrisJobke@skjnldsv