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

[Web] Update Auth Connectors page design #49871

Merged
merged 1 commit into from
Dec 30, 2024

Conversation

rudream
Copy link
Contributor

@rudream rudream commented Dec 6, 2024

Purpose

Part of https://github.com/gravitational/teleport.e/issues/4989

e counterpart: https://github.com/gravitational/teleport.e/pull/5640

This PR updates the design of the auth connectors page, in preparation for additional customizability options.

This also adds shortcuts for adding Okta and Entra ID auth connectors, which will redirect the user to their respective integration enrolment pages as opposed to the YAML editor. Additionally, certain auth connector tiles for certain IdP's (eg. Okta, Entra ID, GitLab, Auth0) will now have their own special icons (detected based on name).

Figma mockups

Demo

OSS empty state

image

OSS w/ connectors

image

Enterprise empty state

image

Enterprise w/ connectors

image

@rudream rudream added no-changelog Indicates that a PR does not require a changelog entry backport/branch/v17 labels Dec 6, 2024
@github-actions github-actions bot requested review from kiosion and ravicious December 6, 2024 11:18
@ravicious ravicious self-requested a review December 12, 2024 11:59
@ravicious
Copy link
Member

ravicious commented Dec 12, 2024

Oops, didn't see that you're yet to commit the changes. Request a review from me please once you get around to that.

@ravicious ravicious removed their request for review December 12, 2024 12:00
@rudream
Copy link
Contributor Author

rudream commented Dec 12, 2024

Oops, didn't see that you're yet commit the changes. Request a review from me please once you get around to that.

@ravicious The UX team has changed the designs quite a bit since I opened this PR, so I'll update this with the new designs (which will change a good chunk of the implementation)

@rudream rudream force-pushed the yassine/auth-connectors-redesign branch 2 times, most recently from 1516325 to 4dfeb47 Compare December 20, 2024 10:03
@rudream rudream requested a review from ravicious December 20, 2024 10:03
@rudream rudream force-pushed the yassine/auth-connectors-redesign branch from 4dfeb47 to 3e0fc4c Compare December 20, 2024 10:31
@rudream rudream force-pushed the yassine/auth-connectors-redesign branch from 3e0fc4c to 39cb96e Compare December 30, 2024 19:56
@rudream rudream enabled auto-merge December 30, 2024 19:56
@rudream rudream added this pull request to the merge queue Dec 30, 2024
Merged via the queue into master with commit f2701af Dec 30, 2024
41 checks passed
@rudream rudream deleted the yassine/auth-connectors-redesign branch December 30, 2024 20:13
@public-teleport-github-review-bot

@rudream See the table below for backport results.

Branch Result
branch/v17 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/branch/v17 no-changelog Indicates that a PR does not require a changelog entry size/md ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants