Skip to content

Commit e70ad2b

Browse files
authored
fix: always show upload and scratch in create template gallery (#14327)
1 parent 5246f8d commit e70ad2b

File tree

7 files changed

+20
-69
lines changed

7 files changed

+20
-69
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 & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,13 @@
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 { showOrganizations } = 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.
@@ -22,17 +19,10 @@ const CreateTemplatesGalleryPage: FC = () => {
2219
<Helmet>
2320
<title>{pageTitle("Create a Template")}</title>
2421
</Helmet>
25-
{showOrganizations ? (
26-
<CreateTemplatesPageView
27-
error={templateExamplesQuery.error}
28-
starterTemplatesByTag={starterTemplatesByTag}
29-
/>
30-
) : (
31-
<StarterTemplatesPageView
32-
error={templateExamplesQuery.error}
33-
starterTemplatesByTag={starterTemplatesByTag}
34-
/>
35-
)}
22+
<CreateTemplateGalleryPageView
23+
error={templateExamplesQuery.error}
24+
starterTemplatesByTag={starterTemplatesByTag}
25+
/>
3626
</>
3727
);
3828
};

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"),
@@ -359,7 +359,7 @@ export const router = createBrowserRouter(
359359
<Route path="/workspaces" element={<WorkspacesPage />} />
360360

361361
<Route path="/starter-templates">
362-
<Route index element={<CreateTemplatesGalleryPage />} />
362+
<Route index element={<CreateTemplateGalleryPage />} />
363363
<Route path=":exampleId" element={<StarterTemplatePage />} />
364364
</Route>
365365

0 commit comments

Comments
 (0)
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy