Skip to content

Make SSO Configuration UI more clear#3971

Merged
juliusgeo merged 4 commits into
mainfrom
sso_config_ui_fixes
May 20, 2026
Merged

Make SSO Configuration UI more clear#3971
juliusgeo merged 4 commits into
mainfrom
sso_config_ui_fixes

Conversation

@juliusgeo
Copy link
Copy Markdown
Contributor

@juliusgeo juliusgeo commented May 19, 2026

Description

  1. make it such that the SSO configuration pane can be dismissed
  2. add a warning when SSO is config'ed but no domains are set
  3. remove domain list placeholder
  4. remove dependency on posthog for showing SSO page

Fixes # (issue)

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Chore (changes which are not directly related to any business logic)

Checklist

Changes have been:

  • Tested (unit, integration, or manually with steps specified)
  • Linted and formatted
  • Documented (where applicable)
  • Added to CHANGELOG (where applicable) -- see Keep a Changelog
image
🤖 AI Disclosure
  • I acknowledge that an LLM was used in the creation of this Pull Request, in accordance with Hatchet's AI_POLICY.md.
  • Details: Claude code, used to generate the frontend changes quickly.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 19, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
hatchet-docs Ready Ready Preview, Comment May 20, 2026 4:26pm

Request Review

@promptless-for-oss
Copy link
Copy Markdown

Promptless prepared a documentation update related to this change.

Triggered by PR #3971

Added documentation for the SSO domain verification workflow, explaining how to enable automatic SSO redirect for organization members by adding and verifying domains via DNS TXT records. This complements the UI clarifications in this PR.

Review: Document Enterprise SSO configuration

@juliusgeo juliusgeo merged commit c972091 into main May 20, 2026
26 of 27 checks passed
@juliusgeo juliusgeo deleted the sso_config_ui_fixes branch May 20, 2026 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants