Page MenuHomePhabricator

Fix redirect to Password settings panel after "Reset Password" login
ClosedPublic

Authored by epriestley on Jun 7 2016, 9:06 PM.
Tags
None
Referenced Files
F13171521: D16075.diff
Tue, May 7, 11:11 AM
Unknown Object (File)
Tue, Apr 30, 5:40 AM
Unknown Object (File)
Thu, Apr 25, 10:23 PM
Unknown Object (File)
Sat, Apr 20, 3:24 PM
Unknown Object (File)
Wed, Apr 10, 9:17 PM
Unknown Object (File)
Tue, Apr 9, 5:33 AM
Unknown Object (File)
Apr 7 2024, 12:00 PM
Unknown Object (File)
Feb 8 2024, 6:19 PM
Subscribers
None

Details

Summary

Fixes T11107. The URI change here meant we were dropping the "key" parameter, which allows you to set a new password without knowing your old one.

Test Plan

Reset password, didn't need to provide old one anymore.

Diff Detail

Repository
rP Phabricator
Branch
ppanel
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 12550
Build 15922: Run Core Tests
Build 15921: arc lint + arc unit

Unit TestsFailed

TimeTest
79 msDifferentialParseRenderTestCase::Unknown Unit Message ("")
Assertion failed, expected values to be equal (at DifferentialParseRenderTestCase.php:62): move.diff.one.expect Expected vs Actual Output Diff --- Old Value
0 msAlmanacNamesTestCase::Unknown Unit Message ("")
30 assertions passed.
0 msAlmanacServiceTypeTestCase::Unknown Unit Message ("")
1 assertion passed.
0 msAphrontHTTPSinkTestCase::Unknown Unit Message ("")
3 assertions passed.
0 msAphrontHTTPSinkTestCase::Unknown Unit Message ("")
1 assertion passed.
View Full Test Results (1 Failed · 321 Passed)

Event Timeline

epriestley retitled this revision from to Fix redirect to Password settings panel after "Reset Password" login.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Jun 7 2016, 9:06 PM
This revision was automatically updated to reflect the committed changes.