Skip to content

Commit 2ed36da

Browse files
committed
fix: always show upload and scratch in create template gallery
1 parent 2c150d0 commit 2ed36da

7 files changed

+20
-70
lines changed

site/src/pages/CreateTemplatesGalleryPage/StarterTemplatesPage.test.tsx renamed to site/src/pages/CreateTemplateGalleryPage/CreateTemplateGalleryPage.test.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,11 @@ import {
88
MockTemplateExample2,
99
} from "testHelpers/entities";
1010
import { server } from "testHelpers/server";
11-
import StarterTemplatesPage from "./CreateTemplatesGalleryPage";
11+
import CreateTemplateGalleryPage from "./CreateTemplateGalleryPage";
1212

1313
test("does not display the scratch template", async () => {
1414
server.use(
15-
http.get("api/v2/organizations/:organizationId/templates/examples", () => {
15+
http.get("api/v2/templates/examples", () => {
1616
return HttpResponse.json([
1717
MockTemplateExample,
1818
MockTemplateExample2,
@@ -36,7 +36,7 @@ test("does not display the scratch template", async () => {
3636
children: [
3737
{
3838
path: "/starter-templates",
39-
element: <StarterTemplatesPage />,
39+
element: <CreateTemplateGalleryPage />,
4040
},
4141
],
4242
},

site/src/pages/CreateTemplatesGalleryPage/CreateTemplatesGalleryPage.tsx renamed to site/src/pages/CreateTemplateGalleryPage/CreateTemplateGalleryPage.tsx

Lines changed: 5 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,39 +1,28 @@
11
import { templateExamples } from "api/queries/templates";
22
import type { TemplateExample } from "api/typesGenerated";
3-
import { useDashboard } from "modules/dashboard/useDashboard";
43
import type { FC } from "react";
54
import { Helmet } from "react-helmet-async";
65
import { useQuery } from "react-query";
76
import { pageTitle } from "utils/page";
87
import { getTemplatesByTag } from "utils/starterTemplates";
9-
import { CreateTemplatesPageView } from "./CreateTemplatesPageView";
10-
import { StarterTemplatesPageView } from "./StarterTemplatesPageView";
8+
import { CreateTemplateGalleryPageView } from "./CreateTemplateGalleryPageView";
119

1210
const CreateTemplatesGalleryPage: FC = () => {
13-
const { experiments } = useDashboard();
1411
const templateExamplesQuery = useQuery(templateExamples());
1512
const starterTemplatesByTag = templateExamplesQuery.data
1613
? // Currently, the scratch template should not be displayed on the starter templates page.
1714
getTemplatesByTag(removeScratchExample(templateExamplesQuery.data))
1815
: undefined;
19-
const multiOrgExperimentEnabled = experiments.includes("multi-organization");
2016

2117
return (
2218
<>
2319
<Helmet>
2420
<title>{pageTitle("Create a Template")}</title>
2521
</Helmet>
26-
{multiOrgExperimentEnabled ? (
27-
<CreateTemplatesPageView
28-
error={templateExamplesQuery.error}
29-
starterTemplatesByTag={starterTemplatesByTag}
30-
/>
31-
) : (
32-
<StarterTemplatesPageView
33-
error={templateExamplesQuery.error}
34-
starterTemplatesByTag={starterTemplatesByTag}
35-
/>
36-
)}
22+
<CreateTemplateGalleryPageView
23+
error={templateExamplesQuery.error}
24+
starterTemplatesByTag={starterTemplatesByTag}
25+
/>
3726
</>
3827
);
3928
};

site/src/pages/CreateTemplatesGalleryPage/StarterTemplatesPageView.stories.tsx renamed to site/src/pages/CreateTemplateGalleryPage/CreateTemplateGalleryPageView.stories.tsx

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6,16 +6,16 @@ import {
66
mockApiError,
77
} from "testHelpers/entities";
88
import { getTemplatesByTag } from "utils/starterTemplates";
9-
import { StarterTemplatesPageView } from "./StarterTemplatesPageView";
9+
import { CreateTemplateGalleryPageView } from "./CreateTemplateGalleryPageView";
1010

11-
const meta: Meta<typeof StarterTemplatesPageView> = {
12-
title: "pages/StarterTemplatesPage",
11+
const meta: Meta<typeof CreateTemplateGalleryPageView> = {
12+
title: "pages/CreateTemplateGalleryPage",
1313
parameters: { chromatic },
14-
component: StarterTemplatesPageView,
14+
component: CreateTemplateGalleryPageView,
1515
};
1616

1717
export default meta;
18-
type Story = StoryObj<typeof StarterTemplatesPageView>;
18+
type Story = StoryObj<typeof CreateTemplateGalleryPageView>;
1919

2020
export const Example: Story = {
2121
args: {

site/src/pages/CreateTemplatesGalleryPage/CreateTemplatesPageView.tsx renamed to site/src/pages/CreateTemplateGalleryPage/CreateTemplateGalleryPageView.tsx

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -13,15 +13,14 @@ import { Link as RouterLink } from "react-router-dom";
1313
import type { StarterTemplatesByTag } from "utils/starterTemplates";
1414
import { StarterTemplates } from "./StarterTemplates";
1515

16-
export interface CreateTemplatePageViewProps {
16+
export interface CreateTemplateGalleryPageViewProps {
1717
starterTemplatesByTag?: StarterTemplatesByTag;
1818
error?: unknown;
1919
}
2020

21-
export const CreateTemplatesPageView: FC<CreateTemplatePageViewProps> = ({
22-
starterTemplatesByTag,
23-
error,
24-
}) => {
21+
export const CreateTemplateGalleryPageView: FC<
22+
CreateTemplateGalleryPageViewProps
23+
> = ({ starterTemplatesByTag, error }) => {
2524
return (
2625
<Margins>
2726
<PageHeader>

site/src/pages/CreateTemplatesGalleryPage/StarterTemplatesPageView.tsx

Lines changed: 0 additions & 38 deletions
This file was deleted.

site/src/router.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -169,8 +169,8 @@ const TemplateVersionPage = lazy(
169169
const TemplateVersionEditorPage = lazy(
170170
() => import("./pages/TemplateVersionEditorPage/TemplateVersionEditorPage"),
171171
);
172-
const CreateTemplatesGalleryPage = lazy(
173-
() => import("./pages/CreateTemplatesGalleryPage/CreateTemplatesGalleryPage"),
172+
const CreateTemplateGalleryPage = lazy(
173+
() => import("./pages/CreateTemplateGalleryPage/CreateTemplateGalleryPage"),
174174
);
175175
const StarterTemplatePage = lazy(
176176
() => import("pages/StarterTemplatePage/StarterTemplatePage"),
@@ -353,7 +353,7 @@ export const router = createBrowserRouter(
353353
<Route path="/workspaces" element={<WorkspacesPage />} />
354354

355355
<Route path="/starter-templates">
356-
<Route index element={<CreateTemplatesGalleryPage />} />
356+
<Route index element={<CreateTemplateGalleryPage />} />
357357
<Route path=":exampleId" element={<StarterTemplatePage />} />
358358
</Route>
359359

0 commit comments

Comments
 (0)