Skip to content

WebDiscover: Create connect to aws account screen#24506

Merged
kimlisa merged 5 commits intomasterfrom
lisa/connect-aws-account-screen
Apr 13, 2023
Merged

WebDiscover: Create connect to aws account screen#24506
kimlisa merged 5 commits intomasterfrom
lisa/connect-aws-account-screen

Conversation

@kimlisa
Copy link
Copy Markdown
Contributor

@kimlisa kimlisa commented Apr 13, 2023

part of #22129

This feature is not enabled. Other pieces of work are coming.

This is the screen that user gets to after selecting Amazon RDS. Then user either selects from existing AWS integrations or create a new one (leads to integration enroll screen)

Screenshot

when user doesn't have any aws integrations
image

when user has existing aws integrations (ignore the aws region it's changed to aws integrations
image

when erroring
image

when user does not have access
image

@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from rudream April 13, 2023 15:53
@kimlisa kimlisa force-pushed the lisa/connect-aws-account-screen branch from a7cf451 to 9a20fc5 Compare April 13, 2023 21:09
@kimlisa kimlisa enabled auto-merge April 13, 2023 21:10
@kimlisa kimlisa added this pull request to the merge queue Apr 13, 2023
Merged via the queue into master with commit e8e9ad3 Apr 13, 2023
@kimlisa kimlisa deleted the lisa/connect-aws-account-screen branch April 13, 2023 21:30
kimlisa added a commit that referenced this pull request May 4, 2023
* WebDiscover: Enroll New AWS Integration Flow (#24459)

* Add Lock icon

* WebDiscover: Create connect to aws account screen (#24506)

* WebDiscover: Add delete integration operation (#24590)

* Fix malformed JSON error response for 200

* Fix making response when fetching integrations

* Create re-usable integration ops hook (only delete for now)

* Create delete dialog

* Allow deleting integrations

* Fix lint
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.

3 participants