-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Uptake TOTP and recovery code changes and docs #294
Conversation
dist/b2b/organizations_members.js
Outdated
* @throws A {@link StytchError} on a non-2xx response from the Stytch API | ||
* @throws A {@link RequestError} when the Stytch API cannot be reached | ||
*/ | ||
totp(data, options) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
deleteTOTP? Also documentation?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah this one is super strange. even the object names are missing the DeleteMFA
portion that is present in the protos. I can manually update but sdk-codegen might just wipe it again unless we can figure out why its like this. I think I should just manually rename, and then look into how why its not updating the same way the delete otp phone number endpoint
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@taronish-stytch - I can work with you tomorrow to figure out why codegen is doing weird things.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good besides totp
--> deleteTotp
No description provided.