Skip to content

allow using saved login credentials for notify - #23021

Merged
blizzz merged 2 commits into
masterfrom
notify-saved-login-auth
Oct 13, 2020
Merged

allow using saved login credentials for notify#23021
blizzz merged 2 commits into
masterfrom
notify-saved-login-auth

Conversation

@icewind1991

Copy link
Copy Markdown
Member

when only the user is provided, we use to provided user to try and get saved login credentials

Signed-off-by: Robin Appelman robin@icewind.nl

@icewind1991icewind1991 added the 3. to review Waiting for reviews label Sep 23, 2020
@icewind1991icewind1991 added this to the Nextcloud 21 milestone Sep 23, 2020
when only the user is provided, we use to provided user to try and get saved login credentials
Signed-off-by: Robin Appelman <robin@icewind.nl>

@rullzerrullzer 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.

🐘

Signed-off-by: Robin Appelman <robin@icewind.nl>

@blizzzblizzz 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.

looks good

@blizzz
blizzz merged commit 060286c into masterOct 13, 2020
@blizzz
blizzz deleted the notify-saved-login-auth branch October 13, 2020 17:39
@blizzz

Copy link
Copy Markdown
Member

/backport to stable20

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@icewind1991@blizzz@rullzer@MorrisJobke