Skip to content

feat: add warning message when trying to delete active template #10142

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.

Sign up for GitHub

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 10 commits into from
Oct 10, 2023
Merged

feat: add warning message when trying to delete active template #10142

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
9b1ae14
refactor: clean up TemplatePageHeader
Parkreiner Oct 9, 2023
3349176
chore: add react query configs for workspace lists
Parkreiner Oct 9, 2023
5604a0d
feat: add delete-intercept functionality
Parkreiner Oct 9, 2023
8f50beb
refactor: improve readability
Parkreiner Oct 9, 2023
eac4164
refactor: rename entities for readability/accuracy
Parkreiner Oct 9, 2023
1cc2935
refactor: clean up variable names again
Parkreiner Oct 9, 2023
8b6653d
refactor: remove redudant function calls
Parkreiner Oct 9, 2023
164227e
fix: update logic check for safe deletions
Parkreiner Oct 9, 2023
bf2394e
fix: update workspaces query logic
Parkreiner Oct 9, 2023
fd220d9
fix: update call site for workspaces key
Parkreiner Oct 9, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
fix: update call site for workspaces key
  • Loading branch information
@Parkreiner
Parkreiner committed Oct 9, 2023
commit fd220d980654b33caec26fc35b3ad142d30ed1ef
4 changes: 2 additions & 2 deletions site/src/pages/TemplatePage/TemplatePageHeader.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ import { Link as RouterLink, useNavigate } from "react-router-dom";
import { useDeletionDialogState } from "./useDeletionDialogState";

import { useQuery } from "react-query";
import { workspacesByQuery } from "api/queries/workspaces";
import { workspaces } from "api/queries/workspaces";
import { ConfirmDialog } from "components/Dialogs/ConfirmDialog/ConfirmDialog";
import {
AuthorizationResponse,
Expand Down Expand Up @@ -52,7 +52,7 @@ const TemplateMenu: FC<TemplateMenuProps> = ({

const queryText = `template:${templateName}`;
const workspaceCountQuery = useQuery({
...workspacesByQuery(queryText),
...workspaces({ q: queryText }),
select: (res) => res.count,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TIL!

});

Expand Down