Skip to content

Hide SSO section account settings if no providers are configured #9444

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

Closed
bpmct opened this issue Aug 30, 2023 · 3 comments · Fixed by #9818
Closed

Hide SSO section account settings if no providers are configured #9444

bpmct opened this issue Aug 30, 2023 · 3 comments · Fixed by #9818
Assignees
Labels
good first issue Easily solved issues suitable for starters and community contributors s3 Bugs that confuse, annoy, or are purely cosmetic site Area: frontend dashboard

Comments

@bpmct
Copy link
Member

bpmct commented Aug 30, 2023

Alternatively, we could advertise the feature and say: "No SSO providers are configured with this Coder deployment".

image

@bpmct bpmct added the s3 Bugs that confuse, annoy, or are purely cosmetic label Aug 30, 2023
@sharkymark
Copy link
Contributor

Related to #9394 @bpmct

@cdr-bot cdr-bot bot added feature labels Aug 30, 2023
@matifali matifali removed the feature label Aug 31, 2023
@BrunoQuaresma BrunoQuaresma added site Area: frontend dashboard good first issue Easily solved issues suitable for starters and community contributors labels Sep 5, 2023
@Parkreiner Parkreiner self-assigned this Sep 21, 2023
@Parkreiner
Copy link
Member

Taking this one.

@Parkreiner
Copy link
Member

Have a preview:
Screenshot 2023-09-21 at 5 11 37 PM

Going to be opening a PR with the changes in just a few minutes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Easily solved issues suitable for starters and community contributors s3 Bugs that confuse, annoy, or are purely cosmetic site Area: frontend dashboard
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants