Skip to content
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

type script definition for sendMFACode is incorrect #5144

Closed
mk0sojo opened this issue Mar 19, 2020 · 1 comment · Fixed by #5145
Closed

type script definition for sendMFACode is incorrect #5144

mk0sojo opened this issue Mar 19, 2020 · 1 comment · Fixed by #5145
Assignees
Labels
Auth Related to Auth components/category to-be-reproduced Used in order for Amplify to reproduce said issue

Comments

@mk0sojo
Copy link
Contributor

mk0sojo commented Mar 19, 2020

Describe the bug
The typecript definition for cognitoUser.sendMFACode is missing parameter clientMetadata.

To Reproduce
Create a typescript file and try to call cognitoUser.sendMFACode passing clientMetadata. You will get an error

Minor change, I will raise a pull request

@mk0sojo mk0sojo added the to-be-reproduced Used in order for Amplify to reproduce said issue label Mar 19, 2020
@sammartinez sammartinez added the Auth Related to Auth components/category label Mar 19, 2020
@ericclemmons ericclemmons linked a pull request Mar 23, 2020 that will close this issue
@github-actions
Copy link

This issue has been automatically locked since there hasn't been any recent activity after it was closed. Please open a new issue for related bugs.

Looking for a help forum? We recommend joining the Amplify Community Discord server *-help channels or Discussions for those types of questions.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Auth Related to Auth components/category to-be-reproduced Used in order for Amplify to reproduce said issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants