Skip to content

Send new password instead of empty string - #16781

Merged
rullzer merged 3 commits into
masterfrom
bug/fix-password-reset
Aug 19, 2019
Merged

Send new password instead of empty string#16781
rullzer merged 3 commits into
masterfrom
bug/fix-password-reset

Conversation

@kesselb

Copy link
Copy Markdown
Contributor

Password reset does not work for me on master.

@kesselbkesselb added bug 3. to review Waiting for reviews labels Aug 18, 2019
@kesselbkesselb added this to the Nextcloud 17 milestone Aug 18, 2019
@kesselb

Copy link
Copy Markdown
ContributorAuthor

After a successful password reset i see the login form but the url looks weird. A login form submit leads to a http exception sometimes because it tries to open the password reset page again. Can someone with more vue knowledge please have a look?

@juliusknorrjuliusknorr added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Aug 18, 2019
@juliusknorr

Copy link
Copy Markdown
Member

Let me have a look then

@rullzerrullzer mentioned this pull request Aug 18, 2019
kesselband others added 3 commits August 19, 2019 09:32
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@juliusknorr
juliusknorrforce-pushed the bug/fix-password-reset branch from 88f835d to c2f039fCompareAugust 19, 2019 07:37
@juliusknorr

Copy link
Copy Markdown
Member

@kesselb Should be fine now.

@juliusknorrjuliusknorr added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Aug 19, 2019
@skjnldsvskjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Aug 19, 2019
@rullzer
rullzer merged commit b73df01 into masterAug 19, 2019
@rullzer
rullzer deleted the bug/fix-password-reset branch August 19, 2019 09:53
@kesselb

Copy link
Copy Markdown
ContributorAuthor

@kesselb Should be fine now.

Looks good 👍

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

Labels

4. to releaseReady to be released and/or waiting for tests to finishbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@kesselb@juliusknorr@skjnldsv@rullzer