Skip to content

chore: consolidate ManageSettingsLayout code #14885

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 17 commits into from
Oct 3, 2024
Prev Previous commit
Next Next commit
oih
  • Loading branch information
aslilac committed Oct 2, 2024
commit c8eaa77f07fe69ffedb7f1b17d0614e721e68818
12 changes: 6 additions & 6 deletions site/src/modules/management/SidebarView.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -147,12 +147,12 @@ const DeploymentSettingsNavigation: FC<DeploymentSettingsNavigationProps> = ({
Users
</SidebarNavSubItem>
)}
<Stack direction="row" alignItems="center" spacing={0}>
<SidebarNavSubItem href="notifications">
Notifications
</SidebarNavSubItem>
<FeatureStageBadge contentType="beta" size="sm" />
</Stack>
<SidebarNavSubItem href="notifications">
<Stack direction="row" alignItems="center" spacing={1}>
<span>Notifications</span>
<FeatureStageBadge contentType="beta" size="sm" />
</Stack>
</SidebarNavSubItem>
</Stack>
)}
</div>
Expand Down
62 changes: 0 additions & 62 deletions site/src/pages/ManagementSettingsPage/Sidebar.tsx

This file was deleted.

225 changes: 0 additions & 225 deletions site/src/pages/ManagementSettingsPage/SidebarView.stories.tsx

This file was deleted.

Loading
Loading