-
Notifications
You must be signed in to change notification settings - Fork 202
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
feat(Auth): Rename resendConfirmationCode API for User Attributes to sendVerificationCode #3384
Conversation
Codecov ReportAttention:
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #3384 +/- ##
==========================================
- Coverage 68.17% 67.88% -0.29%
==========================================
Files 1077 1080 +3
Lines 35973 36025 +52
==========================================
- Hits 24523 24454 -69
- Misses 11450 11571 +121
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
...ins/Auth/Sources/AWSCognitoAuthPlugin/ClientBehavior/AWSCognitoAuthPlugin+UserBehavior.swift
Show resolved
Hide resolved
...AWSCognitoAuthPluginUnitTests/TaskTests/UserBehaviourTests/UserBehaviorResendCodeTests.swift
Show resolved
Hide resolved
Co-authored-by: Michael Law <[email protected]>
@lawmicha Updated the deprecation message. |
Description
To make API behaviour consistent across Amplify Libraries.
Auth.resendConfirmationCode
is being renamed toAuth.sendVerificationCode
. Changes:Auth.resendConfirmationCode
Auth.sendVerificationCode
which is the same as the API being deprecatedGeneral Checklist
Given When Then
inline code documentation and are named accordinglytestThing_condition_expectation()
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.