[nexus] include oxide-scim- prefix in SCIM token response#9301
Merged
david-crespo merged 1 commit intomainfrom Oct 29, 2025
Merged
[nexus] include oxide-scim- prefix in SCIM token response#9301david-crespo merged 1 commit intomainfrom
oxide-scim- prefix in SCIM token response#9301david-crespo merged 1 commit intomainfrom
Conversation
jmpesp
approved these changes
Oct 29, 2025
david-crespo
added a commit
to oxidecomputer/console
that referenced
this pull request
Oct 29, 2025
* Add SCIM tab to UI * Add tests * Tweak truncation * Use 90 day default for expiration time in mock data * revert UI copy change * TODOs update * Remove Delete All from UI and set handler to NotImplemented * npm i * update npm and re-run npm i * Remove todo list * Update import and className ordering * Update path snapshots * tweak height of token pseudo input; order tokens by created_at * Remove EXPIRES column and content from confirmation modal * Simplify mock handler * Revert "Remove EXPIRES column and content from confirmation modal" This reverts commit 47e43af. * Set mock token data to have null expiration datetime * Hide 'Learn about SCIM Auth' text until we have documentation up that we can link to * Remove unnecessary test * For desktops, give a min-width to the Expires column so it doesn't feel so squeezed * SCIM UI tweaks * Fix funky modal (unrelated to SCIM) * Message spacing tweak * Unify input label with others * tweak copy, add saml_scim option in silo create * put oxide-scim- back in. see oxidecomputer/omicron#9301 * fix e2e failure and bug around admin group name * take out the docs link so we can merge --------- Co-authored-by: David Crespo <david.crespo@oxidecomputer.com> Co-authored-by: Benjamin Leonard <benji@oxide.computer>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #9300