Skip to content

Commit 932f829

Browse files
committed
:)
1 parent 16f6650 commit 932f829

File tree

4 files changed

+19
-23
lines changed

4 files changed

+19
-23
lines changed

site/src/modules/management/ManagementSettingsLayout.tsx

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ import { Outlet, useParams } from "react-router-dom";
1414
import { Sidebar } from "./Sidebar";
1515

1616
export const ManagementSettingsContext = createContext<
17-
{ deploymentValues: DeploymentConfig } | undefined
17+
ManagementSettingsValue | undefined
1818
>(undefined);
1919

2020
type ManagementSettingsValue = Readonly<{
@@ -30,21 +30,8 @@ export const useManagementSettings = (): ManagementSettingsValue => {
3030
"useManagementSettings should be used inside of ManagementSettingsLayout",
3131
);
3232
}
33-
const { organizations } = useDashboard();
34-
const { organization: orgName } = useParams() as {
35-
organization?: string;
36-
};
37-
38-
const organization =
39-
organizations && orgName
40-
? organizations.find((org) => org.name === orgName)
41-
: undefined;
4233

43-
return {
44-
deploymentValues: context.deploymentValues,
45-
organizations,
46-
organization,
47-
};
34+
return context;
4835
};
4936

5037
/**
@@ -70,6 +57,10 @@ export const canEditOrganization = (
7057
export const ManagementSettingsLayout: FC = () => {
7158
const { permissions } = useAuthenticated();
7259
const deploymentConfigQuery = useQuery(deploymentConfig());
60+
const { organizations } = useDashboard();
61+
const { organization: orgName } = useParams() as {
62+
organization?: string;
63+
};
7364

7465
// The deployment settings page also contains users, audit logs, groups and
7566
// organizations, so this page must be visible if you can see any of these.
@@ -87,10 +78,19 @@ export const ManagementSettingsLayout: FC = () => {
8778
return <Loader />;
8879
}
8980

81+
const organization =
82+
organizations && orgName
83+
? organizations.find((org) => org.name === orgName)
84+
: undefined;
85+
9086
return (
9187
<RequirePermission isFeatureVisible={canViewDeploymentSettingsPage}>
9288
<ManagementSettingsContext.Provider
93-
value={{ deploymentValues: deploymentConfigQuery.data }}
89+
value={{
90+
deploymentValues: deploymentConfigQuery.data,
91+
organizations,
92+
organization,
93+
}}
9494
>
9595
<Margins>
9696
<Stack css={{ padding: "48px 0" }} direction="row" spacing={6}>

site/src/pages/DeploymentSettingsPage/NotificationsPage/NotificationEvents.stories.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import { NotificationEvents } from "./NotificationEvents";
88
import { baseMeta } from "./storybookUtils";
99

1010
const meta: Meta<typeof NotificationEvents> = {
11-
title: "pages/DeploymentSettings/NotificationsPage/NotificationEvents",
11+
title: "pages/DeploymentSettingsPage/NotificationsPage/NotificationEvents",
1212
component: NotificationEvents,
1313
args: {
1414
defaultMethod: "smtp",

site/src/pages/DeploymentSettingsPage/NotificationsPage/NotificationsPage.stories.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ import { NotificationsPage } from "./NotificationsPage";
1212
import { baseMeta } from "./storybookUtils";
1313

1414
const meta: Meta<typeof NotificationsPage> = {
15-
title: "pages/DeploymentSettings/NotificationsPage",
15+
title: "pages/DeploymentSettingsPage/NotificationsPage",
1616
component: NotificationsPage,
1717
...baseMeta,
1818
};

site/src/pages/DeploymentSettingsPage/OAuth2AppsSettingsPage/CreateOAuth2AppPageView.stories.tsx

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,4 @@ export const WithError: Story = {
3838
},
3939
};
4040

41-
export const Default: Story = {
42-
args: {
43-
// Nothing.
44-
},
45-
};
41+
export const Default: Story = {};

0 commit comments

Comments
 (0)