fix: show org summary page if not entitled

code-asher · code-asher · commit 8b74b1219d0a · 2024-08-16T16:14:34.000-08:00
You cannot edit the settings without being entitled, so show the summary
page instead.
diff --git a/site/src/pages/ManagementSettingsPage/OrganizationSettingsPage.stories.tsx b/site/src/pages/ManagementSettingsPage/OrganizationSettingsPage.stories.tsx
@@ -10,6 +10,7 @@ const meta: Meta<typeof OrganizationSettingsPage> = {
 	decorators: [withAuthProvider, withDashboardProvider],
 	parameters: {
 		user: MockUser,
+		features: ["multiple_organizations"],
 		permissions: { viewDeploymentValues: true },
 		queries: [
 			{
@@ -61,3 +62,23 @@ export const CanEditOrganization: Story = {
 		],
 	},
 };
+
+export const CanEditOrganizationNotEntitled: Story = {
+	parameters: {
+		reactRouter: reactRouterParameters({
+			location: { pathParams: { organization: MockDefaultOrganization.name } },
+			routing: { path: "/organizations/:organization" },
+		}),
+		features: [],
+		queries: [
+			{
+				key: ["organizations", [MockDefaultOrganization.id], "permissions"],
+				data: {
+					[MockDefaultOrganization.id]: {
+						editOrganization: true,
+					},
+				},
+			},
+		],
+	},
+};
diff --git a/site/src/pages/ManagementSettingsPage/OrganizationSettingsPage.tsx b/site/src/pages/ManagementSettingsPage/OrganizationSettingsPage.tsx
@@ -7,6 +7,7 @@ import type { Organization } from "api/typesGenerated";
 import { EmptyState } from "components/EmptyState/EmptyState";
 import { displaySuccess } from "components/GlobalSnackbar/utils";
 import { Loader } from "components/Loader/Loader";
+import { useFeatureVisibility } from "modules/dashboard/useFeatureVisibility";
 import type { FC } from "react";
 import { useMutation, useQuery, useQueryClient } from "react-query";
 import { Navigate, useNavigate, useParams } from "react-router-dom";
@@ -22,6 +23,7 @@ const OrganizationSettingsPage: FC = () => {
 		organization?: string;
 	};
 	const { organizations } = useOrganizationSettings();
+	const feats = useFeatureVisibility();
 
 	const navigate = useNavigate();
 	const queryClient = useQueryClient();
@@ -69,7 +71,9 @@ const OrganizationSettingsPage: FC = () => {
 	// The user may not be able to edit this org but they can still see it because
 	// they can edit members, etc.  In this case they will be shown a read-only
 	// summary page instead of the settings form.
-	if (!permissions[organization.id]?.editOrganization) {
+	// Similarly, if the feature is not entitled then the user will not be able to
+	// edit the organization.
+	if (!permissions[organization.id]?.editOrganization || !feats.multiple_organizations) {
 		return <OrganizationSummaryPageView organization={organization} />;
 	}
 
diff --git a/site/src/testHelpers/storybook.tsx b/site/src/testHelpers/storybook.tsx
@@ -24,6 +24,7 @@ export const withDashboardProvider = (
 
 	const entitlements: Entitlements = {
 		...MockEntitlements,
+		has_license: features.length > 0,
 		features: withDefaultFeatures(
 			Object.fromEntries(
 				features.map((feature) => [
