Skip to content

feat: add combobox for selecting claim field value for group/role idp sync #16459

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

Merged
merged 6 commits into from
Feb 7, 2025

Conversation

jaaydenh
Copy link
Contributor

@jaaydenh jaaydenh commented Feb 5, 2025

contributes to coder/internal#330

This is a followup to #16335

This adds a combobox that swaps with the input for the idp group and idp role names when the sync field returns claim values from the claim field values endpoint. If no claim field values are returned then the input remains instead of the combobox.

Screenshot 2025-02-05 at 20 50 48 Screenshot 2025-02-05 at 20 51 00

@jaaydenh jaaydenh self-assigned this Feb 5, 2025
@jaaydenh jaaydenh requested a review from aslilac February 5, 2025 20:51
@jaaydenh jaaydenh changed the title feat: group/roles IDP sync - add combobox to select claim field value when sync field is set feat: add combobox to select claim field value when sync field is set for group/role idp sync Feb 5, 2025
@jaaydenh jaaydenh changed the title feat: add combobox to select claim field value when sync field is set for group/role idp sync feat: add combobox for selecting claim field value when sync field is set for group/role idp sync Feb 5, 2025
@jaaydenh jaaydenh changed the title feat: add combobox for selecting claim field value when sync field is set for group/role idp sync feat: add combobox for selecting claim field value for group/role idp sync Feb 5, 2025
Copy link
Member

@aslilac aslilac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

main thing I would like changed is the use of onSuccess. we're gonna have to get rid of all those one day, I'd rather not make the list longer. 😅

otherwise looks good!

@jaaydenh jaaydenh force-pushed the jaaydenh/groups-roles-autocomplete branch from 97f619d to 447cad7 Compare February 7, 2025 20:37
@jaaydenh jaaydenh requested a review from aslilac February 7, 2025 20:48
@jaaydenh jaaydenh merged commit 7076c4e into main Feb 7, 2025
30 checks passed
@jaaydenh jaaydenh deleted the jaaydenh/groups-roles-autocomplete branch February 7, 2025 21:31
@github-actions github-actions bot locked and limited conversation to collaborators Feb 7, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants