Skip to content

feat: show warning on unrecognized idp org mapping claims #16478

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 15 commits into from
Feb 6, 2025
+113 −91
Merged

feat: show warning on unrecognized idp org mapping claims #16478

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
5c09338
hoot
aslilac Feb 5, 2025
c52982e
feat: show warning on unrecognized idp org mapping claims
aslilac Feb 6, 2025
5634be1
Merge branch 'main' into lilac/idp-sync-missing-given-warning
aslilac Feb 6, 2025
63ca763
🧹
aslilac Feb 6, 2025
e55e0a8
add warning description
aslilac Feb 6, 2025
72a7986
toolptipp
aslilac Feb 6, 2025
eedf797
tweak text colors
aslilac Feb 6, 2025
06ba32d
Update site/src/pages/DeploymentSettingsPage/IdpOrgSyncPage/IdpOrgSyn…
aslilac Feb 6, 2025
6ffcc94
- `toString()`
aslilac Feb 6, 2025
506cca1
polish
aslilac Feb 6, 2025
c91cd1a
🧼
aslilac Feb 6, 2025
97de7f4
rename
aslilac Feb 6, 2025
06715fa
sad moment
aslilac Feb 6, 2025
42ceb1b
cntnd.
aslilac Feb 6, 2025
820cca3
huh
aslilac Feb 6, 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
🧼
  • Loading branch information
@aslilac
aslilac committed Feb 6, 2025
commit c91cd1adc91f9d23b974f648c7f21ddd70fd418a
2 changes: 1 addition & 1 deletion site/src/pages/DeploymentSettingsPage/IdpOrgSyncPage/IdpOrgSyncPage.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
import { getErrorMessage } from "api/errors";
import { deploymentIdpSyncFieldValues } from "api/queries/deployment";
import {
organizationIdpSyncSettings,
patchOrganizationSyncSettings,
Expand All @@ -18,7 +19,6 @@ import { docs } from "utils/docs";
import { pageTitle } from "utils/page";
import { ExportPolicyButton } from "./ExportPolicyButton";
import IdpOrgSyncPageView from "./IdpOrgSyncPageView";
import { deploymentIdpSyncFieldValues } from "api/queries/deployment";

export const IdpOrgSyncPage: FC = () => {
const queryClient = useQueryClient();
Expand Down
14 changes: 7 additions & 7 deletions site/src/pages/DeploymentSettingsPage/IdpOrgSyncPage/IdpOrgSyncPageView.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
import { TooltipProvider } from "@radix-ui/react-tooltip";
import type {
Organization,
OrganizationSyncSettings,
Expand Down Expand Up @@ -28,12 +29,8 @@ import {
MultiSelectCombobox,
type Option,
} from "components/MultiSelectCombobox/MultiSelectCombobox";
import {
Tooltip,
TooltipContent,
TooltipTrigger,
} from "components/Tooltip/Tooltip";
import { Spinner } from "components/Spinner/Spinner";
import { Stack } from "components/Stack/Stack";
import { Switch } from "components/Switch/Switch";
import {
Table,
Expand All @@ -42,15 +39,18 @@ import {
TableHeader,
TableRow,
} from "components/Table/Table";
import {
Tooltip,
TooltipContent,
TooltipTrigger,
} from "components/Tooltip/Tooltip";
import { useFormik } from "formik";
import { Plus, Trash, TriangleAlert } from "lucide-react";
import { type FC, type KeyboardEventHandler, useId, useState } from "react";
import { docs } from "utils/docs";
import { isUUID } from "utils/uuid";
import * as Yup from "yup";
import { OrganizationPills } from "./OrganizationPills";
import { Stack } from "components/Stack/Stack";
import { TooltipProvider } from "@radix-ui/react-tooltip";

interface IdpSyncPageViewProps {
organizationSyncSettings: OrganizationSyncSettings | undefined;
Expand Down