Details

Reviewers

tomek
inka
ashoat

Commits

rCOMMe0aceb05b71e: [native] use useChangeIdentityUserPassword

Summary

this change lets users change their password on the identity service from native

Depends on D10616

Test Plan

successfully registered a password user, manually updated relevant state, changed their password, and then successfully logged in with the new password

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

varun created this revision.Jan 12 2024, 11:31 PM

Harbormaster completed remote builds in B25735: Diff 35584.Jan 12 2024, 11:50 PM

varun requested review of this revision.Jan 12 2024, 11:50 PM

This code looks right with the API as-is, but unfortunately in D10616 I realized that we're missing a step where we confirm the existing password. I think that will require updates here unfortunately

This revision now requires changes to proceed.Jan 13 2024, 6:50 PM

match whitepaper

ashoat accepted this revision.Tue, Jun 18, 10:10 AM

This revision is now accepted and ready to land.Tue, Jun 18, 10:10 AM

fix logic

Harbormaster completed remote builds in B29751: Diff 41458.Tue, Jun 18, 10:26 AM

Harbormaster completed remote builds in B29752: Diff 41459.Tue, Jun 18, 10:30 AM

varun added a child revision: D12468: [web] allow password users registered with identity to change their password from web.Tue, Jun 18, 10:56 AM

improve error handling

Harbormaster completed remote builds in B29762: Diff 41470.Tue, Jun 18, 2:06 PM

Closed by commit rCOMMe0aceb05b71e: [native] use useChangeIdentityUserPassword (authored by varun). · Explain WhyWed, Jun 19, 6:39 AM

This revision was automatically updated to reflect the committed changes.