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

refactor(authenticator)!: use Amplify JS auto sign in #2642

Merged
merged 90 commits into from Nov 1, 2022

Conversation

ErikCH
Copy link
Contributor

@ErikCH ErikCH commented Sep 24, 2022

Auto Sign In

The will update the Authenticator and xState machine to use the auto sign in feature of the Amplify JS library. This will be the first PR in the Email Link Verification changes.

Description of changes

Added in new breaking change based on the Email Link Verification API Breaking change document.

Added a new breaking change that will require a newer version of the Amplify JS library. In addition, a new breaking change was added for users who override handleSignUp. They will need to add autoSignIn to enabled.

This change will accomplish the following

  • Sign up in xState machine will now default to use the autoSign feature enabled to true.
  • New autoSignIn and autoSignIn_failure are added as new hub event listeners
  • handleSignUp function override now requires autoSignIn to enabled
  • Minimum peer dependency for aws-amplify has been set to ^4.3.29 for React, Vue and Angular
  • New migration documentation has been added

After this is merged another PR will be pushed up with the Email Link Verification changes, that are non breaking.

Issue #, if available

Asana

Description of how you validated changes

Added additional E2E and unit tests

Checklist

  • PR description included
  • yarn test passes
  • Tests are updated
  • No side effects or sideEffects field updated
  • Relevant documentation is changed or added (and PR referenced)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@changeset-bot
Copy link

changeset-bot bot commented Sep 24, 2022

🦋 Changeset detected

Latest commit: b458299

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 6 packages
Name Type
@aws-amplify/ui-react Patch
@aws-amplify/ui Patch
@aws-amplify/ui-angular Patch
@aws-amplify/ui-vue Patch
@aws-amplify/ui-react-core Patch
@aws-amplify/ui-react-native Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@ErikCH ErikCH changed the title [WIP - DO NOT MERGE] - Auto Sign In [WIP - DO NOT MERGE] - feat(auto sign in) Sep 24, 2022
@ErikCH ErikCH temporarily deployed to ci September 24, 2022 22:32 Inactive
wlee221
wlee221 previously approved these changes Oct 31, 2022
0618
0618 previously approved these changes Oct 31, 2022
Copy link
Contributor

@0618 0618 left a comment

Choose a reason for hiding this comment

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

:shipit:

@ErikCH ErikCH dismissed stale reviews from 0618 and wlee221 via 7ff9cd7 October 31, 2022 20:37
@ErikCH ErikCH temporarily deployed to ci October 31, 2022 20:56 Inactive
@ErikCH ErikCH temporarily deployed to ci October 31, 2022 20:56 Inactive
@ErikCH ErikCH temporarily deployed to ci October 31, 2022 20:56 Inactive
@wlee221 wlee221 temporarily deployed to ci October 31, 2022 22:16 Inactive
@wlee221 wlee221 temporarily deployed to ci October 31, 2022 22:16 Inactive
@wlee221 wlee221 temporarily deployed to ci October 31, 2022 22:16 Inactive
@wlee221 wlee221 temporarily deployed to ci October 31, 2022 22:16 Inactive
@ErikCH ErikCH requested review from 0618 and wlee221 October 31, 2022 22:36
Copy link
Contributor

@0618 0618 left a comment

Choose a reason for hiding this comment

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

:shipit:

@wlee221
Copy link
Contributor

wlee221 commented Nov 1, 2022

Tiny little nit before we merge -- let's rename the title / commit to follow conventional commits (e.g. refactor(authenticator)!: use Amplify JS auto sign in)

@ErikCH ErikCH changed the title Breaking Change - feat(auto sign in) refactor(authenticator)!: use Amplify JS auto sign in Nov 1, 2022
@ErikCH
Copy link
Contributor Author

ErikCH commented Nov 1, 2022

Tiny little nit before we merge -- let's rename the title / commit to follow conventional commits (e.g. refactor(authenticator)!: use Amplify JS auto sign in)

Done

@ErikCH ErikCH merged commit 394556a into next-release/main Nov 1, 2022
@ErikCH ErikCH deleted the next-release/auto-sign-in branch November 1, 2022 00:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants