fix: team provider migration

[ammarkarachi]

Issue #, if available:

Description of changes:

The goal of the PR is to change how we mange third party secrets. The secrets are stored in an ephemeral file called ~/.aws/amplify/deployment-secrets.json. On push-resource the auth for which the secrets are deployed. The change in index.ts intercepts incoming commands to find out if the user has to go through migration.

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

[lgtm-com]

This pull request introduces 1 alert when merging bbf8efd into 5d474d3 - view on LGTM.com

new alerts:

• 1 for Unused variable, import, function or class

[lgtm-com]

This pull request introduces 1 alert when merging 6ba672d into 5d474d3 - view on LGTM.com

new alerts:

• 1 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 1 alert when merging 63eeb4c into 7a806b2 - view on LGTM.com

fixed alerts:

• 1 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 1 alert when merging 5915ac9 into c0d73a6 - view on LGTM.com

fixed alerts:

• 1 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 1 alert when merging e3ab873 into c0d73a6 - view on LGTM.com

fixed alerts:

• 1 for Unused variable, import, function or class

[codecov]

Codecov Report

Merging #5733 (def4906) into master (f3df233) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #5733   +/-   ##
=======================================
  Coverage   58.15%   58.15%           
=======================================
  Files         407      407           
  Lines       18749    18749           
  Branches     3747     3747           
=======================================
  Hits        10903    10903           
  Misses       7164     7164           
  Partials      682      682           

Impacted Files	Coverage Δ
.../graphql-transformer-core/src/util/sanity-check.ts	11.29% <0.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f3df233...e5f0a81. Read the comment docs.

[lgtm-com]

This pull request fixes 1 alert when merging f845f01 into c0d73a6 - view on LGTM.com

fixed alerts:

• 1 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 1 alert when merging e970fe3 into 319a3ae - view on LGTM.com

fixed alerts:

• 1 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 1 alert when merging 9b63a22 into 319a3ae - view on LGTM.com

fixed alerts:

• 1 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 1 alert when merging 41fd9a1 into f3df233 - view on LGTM.com

fixed alerts:

• 1 for Unused variable, import, function or class

[kaustavghosh06]

@edwardfoyle as a first step - this is moving all the existing structure which is in team-provider to this file. We can do an API review at a later stage if we want to change the structure. Right now in the migration step we're copy pasting what we have in team-provider to out here and using it only for deployments.

[lgtm-com]

This pull request introduces 1 alert and fixes 1 when merging f929261 into f3df233 - view on LGTM.com

new alerts:

• 1 for Unused variable, import, function or class

fixed alerts:

• 1 for Unused variable, import, function or class

[lgtm-com]

This pull request introduces 4 alerts and fixes 1 when merging 5ebf2af into f3df233 - view on LGTM.com

new alerts:

• 4 for Unused variable, import, function or class

fixed alerts:

• 1 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 1 alert when merging 6509e6b into f3df233 - view on LGTM.com

fixed alerts:

• 1 for Unused variable, import, function or class

[nikhname]

This PR doesn't seem to contain the changes to getCLIPath we discussed. Are we planning on having those in a separate PR?

[ammarkarachi]

This PR doesn't seem to contain the changes to getCLIPath we discussed. Are we planning on having those in a separate PR?
@nikhname

amplify-cli/.circleci/config.base.yml

Line 355 in 6509e6b

AMPLIFY_PATH: /home/circleci/.npm-global/lib/node_modules/@aws-amplify/cli/bin/amplify

we use AMPLIFY_PATH for that

[lgtm-com]

This pull request fixes 1 alert when merging 88d55f2 into f3df233 - view on LGTM.com

fixed alerts:

• 1 for Unused variable, import, function or class

[attilah]

LGTM, thanks for the changes, just update enableExternalAuth as we discussed, and store the root stack id, beside amplifyAppId, in case we need it in the future.

[lgtm-com]

This pull request introduces 1 alert and fixes 1 when merging e5f0a81 into f3df233 - view on LGTM.com

new alerts:

• 1 for Unused variable, import, function or class

fixed alerts:

• 1 for Unused variable, import, function or class

[github-actions]

This pull request 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 for those types of questions.