Skip to content

feat: enable editing of IDP sync configuration for groups and roles in the UI #16098

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.

Sign up for GitHub

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 40 commits into from
Jan 27, 2025
Merged

feat: enable editing of IDP sync configuration for groups and roles in the UI #16098

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
40 commits
Select commit Hold shift + click to select a range
96b0c09
chore: remove stack component
jaaydenh Jan 10, 2025
a88b72f
chore: styling cleanup
jaaydenh Jan 10, 2025
cb00c73
feat: add form and mutation queries
jaaydenh Jan 10, 2025
4b15a2a
feat: add form fields
jaaydenh Jan 12, 2025
0befeca
chore: add delete button
jaaydenh Jan 14, 2025
f8349de
fix: update input component to 40px height
jaaydenh Jan 14, 2025
7124c11
fix: fix styles for MultiSelectCombobox
jaaydenh Jan 14, 2025
30c8a9c
chore: update export policy button design
jaaydenh Jan 14, 2025
a7c954b
fix: update copy and spacing
jaaydenh Jan 14, 2025
c09a3f3
chore: update styles
jaaydenh Jan 14, 2025
8714f5e
fix: fix storybook tests
jaaydenh Jan 14, 2025
b558f35
fix: fix legacy group mappings styles
jaaydenh Jan 14, 2025
aa0e53c
fix: format
jaaydenh Jan 14, 2025
7635856
feat: create link component
jaaydenh Jan 14, 2025
91ca380
fix: use new Link component
jaaydenh Jan 15, 2025
328ce2f
fix: use correct field value
jaaydenh Jan 15, 2025
52d563c
fix: update error handling
jaaydenh Jan 15, 2025
b1d7649
feat: add e2e tests for group and role sync
jaaydenh Jan 15, 2025
b3812f1
chore: update link components
jaaydenh Jan 15, 2025
d19a0dc
fix: remove unnecessary arbitrary values
jaaydenh Jan 16, 2025
43159ff
chore: extract Form components
jaaydenh Jan 16, 2025
528f2d8
fix: adding loading spinners
jaaydenh Jan 16, 2025
d283fa1
fix: update icon button sizing to match design
jaaydenh Jan 17, 2025
ad2fc45
chore: improve e2e tests
jaaydenh Jan 20, 2025
c99e009
fix: change to min width fit
jaaydenh Jan 20, 2025
31cd599
fix: updates for PR review comments
jaaydenh Jan 20, 2025
2354883
fix: remove async from onClick
jaaydenh Jan 22, 2025
3c63c8c
fix: fix tabs styling
jaaydenh Jan 22, 2025
623a50e
fix: format
jaaydenh Jan 22, 2025
5779c8d
fix: updates for PR comments
jaaydenh Jan 22, 2025
395fdc8
chore: switch to useId for form ids
jaaydenh Jan 22, 2025
79ce400
fix: styling
jaaydenh Jan 22, 2025
854684a
fix: add resolves
jaaydenh Jan 22, 2025
50c12db
fix: revert button styles
jaaydenh Jan 22, 2025
16665e2
chore: use Link component
jaaydenh Jan 22, 2025
3dbcee3
fix: show form errors and update mapping schema validation
jaaydenh Jan 27, 2025
9b709ae
fix: remove unnecessary async
jaaydenh Jan 27, 2025
bee5783
fix: remove useEffect
jaaydenh Jan 27, 2025
d3b64b2
fix: error display
jaaydenh Jan 27, 2025
0362c16
fix: format
jaaydenh Jan 27, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: fix storybook tests
  • Loading branch information
@jaaydenh
jaaydenh committed Jan 22, 2025
commit 8714f5ec6a21864a79b405f7fb7a1f4833d2982c
4 changes: 2 additions & 2 deletions site/src/pages/ManagementSettingsPage/IdpSyncPage/ExportPolicyButton.stories.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ export const ClickExportGroupPolicy: Story = {
play: async ({ canvasElement, args }) => {
const canvas = within(canvasElement);
await userEvent.click(
canvas.getByRole("button", { name: "Export Policy" }),
canvas.getByRole("button", { name: "Export policy" }),
);
await waitFor(() =>
expect(args.download).toHaveBeenCalledWith(
Expand All @@ -58,7 +58,7 @@ export const ClickExportRolePolicy: Story = {
play: async ({ canvasElement, args }) => {
const canvas = within(canvasElement);
await userEvent.click(
canvas.getByRole("button", { name: "Export Policy" }),
canvas.getByRole("button", { name: "Export policy" }),
);
await waitFor(() =>
expect(args.download).toHaveBeenCalledWith(
Expand Down
4 changes: 2 additions & 2 deletions site/src/pages/ManagementSettingsPage/IdpSyncPage/IdpSyncPageView.stories.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -83,8 +83,8 @@ export const RolesTab: Story = {
play: async ({ canvasElement }) => {
const user = userEvent.setup();
const canvas = within(canvasElement);
const rolesTab = await canvas.findByText("Role Sync Settings");
const rolesTab = await canvas.findByText("Role sync settings");
await user.click(rolesTab);
await expect(canvas.findByText("IdP Role")).resolves.toBeVisible();
await expect(canvas.findByText("IdP role")).resolves.toBeVisible();
Copy link
Member

Choose a reason for hiding this comment

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

What does resolves do here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It just explicity verifies that the promise from findByText returns correctly but it isn't necessary here as await is already being used. I will remove the resolves.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Actually this test fails if I remove the .resolves

},
};