Skip to content

Commit 4438af8

Browse files
committed
fix: add TooltipProvider to AutoCreateMissingGroupsHelpTooltip and
LegacyGroupSyncHeader
1 parent d30170c commit 4438af8

File tree

1 file changed

+29
-25
lines changed

1 file changed

+29
-25
lines changed

site/src/pages/OrganizationSettingsPage/IdpSyncPage/IdpGroupSyncForm.tsx

Lines changed: 29 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -413,15 +413,17 @@ const GroupRow: FC<GroupRowProps> = ({
413413

414414
const AutoCreateMissingGroupsHelpTooltip: FC = () => {
415415
return (
416-
<HelpTooltip>
417-
<HelpTooltipTrigger />
418-
<HelpTooltipContent>
419-
<HelpTooltipText>
420-
Enabling auto create missing groups will automatically create groups
421-
returned by the OIDC provider if they do not exist in Coder.
422-
</HelpTooltipText>
423-
</HelpTooltipContent>
424-
</HelpTooltip>
416+
<TooltipProvider>
417+
<HelpTooltip>
418+
<HelpTooltipTrigger />
419+
<HelpTooltipContent>
420+
<HelpTooltipText>
421+
Enabling auto create missing groups will automatically create groups
422+
returned by the OIDC provider if they do not exist in Coder.
423+
</HelpTooltipText>
424+
</HelpTooltipContent>
425+
</HelpTooltip>
426+
</TooltipProvider>
425427
);
426428
};
427429

@@ -430,22 +432,24 @@ const LegacyGroupSyncHeader: FC = () => {
430432
<h4 className="text-xl font-medium">
431433
<div className="flex items-end gap-2">
432434
<span>Legacy group sync settings</span>
433-
<HelpTooltip>
434-
<HelpTooltipTrigger />
435-
<HelpTooltipContent>
436-
<HelpTooltipTitle>Legacy group sync settings</HelpTooltipTitle>
437-
<HelpTooltipText>
438-
These settings were configured using environment variables, and
439-
only apply to the default organization. It is now recommended to
440-
configure IdP sync via the CLI or the UI, which enables sync to be
441-
configured for any organization, and for those settings to be
442-
persisted without manually setting environment variables.{" "}
443-
<Link href={docs("/admin/users/idp-sync")}>
444-
Learn more&hellip;
445-
</Link>
446-
</HelpTooltipText>
447-
</HelpTooltipContent>
448-
</HelpTooltip>
435+
<TooltipProvider>
436+
<HelpTooltip>
437+
<HelpTooltipTrigger />
438+
<HelpTooltipContent>
439+
<HelpTooltipTitle>Legacy group sync settings</HelpTooltipTitle>
440+
<HelpTooltipText>
441+
These settings were configured using environment variables, and
442+
only apply to the default organization. It is now recommended to
443+
configure IdP sync via the CLI or the UI, which enables sync to
444+
be configured for any organization, and for those settings to be
445+
persisted without manually setting environment variables.{" "}
446+
<Link href={docs("/admin/users/idp-sync")}>
447+
Learn more&hellip;
448+
</Link>
449+
</HelpTooltipText>
450+
</HelpTooltipContent>
451+
</HelpTooltip>
452+
</TooltipProvider>
449453
</div>
450454
</h4>
451455
);

0 commit comments

Comments
 (0)