Skip to content

Web: fix attempt state with AddApp component#48607

Merged
kimlisa merged 1 commit intomasterfrom
lisa/fix-non-enterprise-manual-add-app
Nov 7, 2024
Merged

Web: fix attempt state with AddApp component#48607
kimlisa merged 1 commit intomasterfrom
lisa/fix-non-enterprise-manual-add-app

Conversation

@kimlisa
Copy link
Copy Markdown
Contributor

@kimlisa kimlisa commented Nov 7, 2024

fixes #48389

In open source, the manual instructions on adding application renders (which still uses a token), and before fully fetching token, it was attempting to read the token which resulted in the bug.

changelog: Fix undefined error in open source version when clicking on Add Application tile in the Enroll Resources page in the Web UI

@aws-amplify-us-west-2
Copy link
Copy Markdown

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-48607.d3pp5qlev8mo18.amplifyapp.com

@kimlisa kimlisa enabled auto-merge November 7, 2024 19:06
@kimlisa kimlisa added this pull request to the merge queue Nov 7, 2024
@kimlisa kimlisa changed the title Web: attempt state with AddApp component Web: fix attempt state with AddApp component Nov 7, 2024
Merged via the queue into master with commit 92ec682 Nov 7, 2024
@kimlisa kimlisa deleted the lisa/fix-non-enterprise-manual-add-app branch November 7, 2024 19:17
@public-teleport-github-review-bot
Copy link
Copy Markdown

@kimlisa See the table below for backport results.

Branch Result
branch/v15 Create PR
branch/v16 Create PR
branch/v17 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Application resource enrollment page doesn't render (WebUI)

3 participants