Skip to content
This repository was archived by the owner on Jun 17, 2022. It is now read-only.

extend functionality for set password flow #150

Merged
merged 1 commit into from
Aug 17, 2020
Merged

Conversation

kspearrin
Copy link
Member

Extend set password models to accept new api properties

update change password base component to work for setting password and other crypto keys for new users via sso

@kspearrin kspearrin requested a review from a team August 14, 2020 21:32
Copy link
Member

@vincentsalucci vincentsalucci left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(obvious) Clarification - for the existing users (ex: Settings flow), you'll just be passing in the kdf and kdIterations in the necessary sub-classed components using the user service beforehand, yes?

@kspearrin
Copy link
Member Author

@vincentsalucci yes, in normal authed states, you can fetch the information from userService. SSO auth doesn't return that when going through the new user flow (since you had no password set yet).

@kspearrin kspearrin merged commit 7bf00b4 into master Aug 17, 2020
@kspearrin kspearrin deleted the setpasswordapi branch August 17, 2020 14:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants