/server

Remove "password reset token" after successful login #1024

Merged
merged 2 commits into from Aug 24, 2016

Labels

3. to review bug

Milestone

  Nextcloud 11.0

Assignees

No one assigned

4 participants

@nickvergessen @mention-bot @rullzer @LukasReschke
@nickvergessen
Nextcloud member
nickvergessen commented Aug 23, 2016

@LukasReschke

@nickvergessen nickvergessen Remove "password reset token" after successful login
This commit was signed with a verified signature.
@nickvergessen nickvergessen Joas Schilling
GPG key ID: E166FD8976B3BAC8 Learn about signing commits

Some checks haven’t completed yet

1 pending and 3 successful checks
approvals/lgtm — this commit is pending approval
Details
Scrutinizer — 1 updated code elements
Details
continuous-integration/drone — the build was successful
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
139fb8d
@nickvergessen nickvergessen added this to the Nextcloud 11.0 milestone Aug 23, 2016
@mention-bot
mention-bot commented Aug 23, 2016

@nickvergessen, thanks for your PR! By analyzing the annotation information on this pull request, we identified @LukasReschke, @ChristophWurst and @DeepDiver1975 to be potential reviewers

@nickvergessen nickvergessen Move the reset token to core app
This commit was signed with a verified signature.
@nickvergessen nickvergessen Joas Schilling
GPG key ID: E166FD8976B3BAC8 Learn about signing commits

All checks have passed

3 successful checks
Details
Scrutinizer — 1 updated code elements
approvals/lgtm — this commit looks good
Details
continuous-integration/drone — the build was successful
736e884
@rullzer
Nextcloud member
rullzer commented Aug 24, 2016

👍

@LukasReschke
Nextcloud member
LukasReschke commented Aug 24, 2016

LGTM

@LukasReschke LukasReschke merged commit a27a8d7 into master Aug 24, 2016
3 checks passed

3 checks passed

Details Scrutinizer 1 updated code elements
approvals/lgtm this commit looks good
Details continuous-integration/drone the build was successful
@LukasReschke LukasReschke deleted the remove-password-reset-link-after-successful-login branch Aug 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment