Skip to content

feat: have user type name of thing to delete for extra safety #4080

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 12 commits into from
Sep 20, 2022
Merged

feat: have user type name of thing to delete for extra safety #4080

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
614fc86
Add info and text field to delete dialog
presleyp Sep 14, 2022
0dc2717
Format
presleyp Sep 14, 2022
cf53085
Use DeleteDialog for Users, nix info except for Workspaces
presleyp Sep 14, 2022
d52354a
Format
presleyp Sep 14, 2022
a8a78f0
Update storybook
presleyp Sep 14, 2022
b8a6c53
Add and update tests
presleyp Sep 15, 2022
ce02bed
Merge branch 'main' into presleyp/destructive-dialog
presleyp Sep 15, 2022
06e6e65
Fix the worst of the UsersPage test bugs
presleyp Sep 15, 2022
2a85c53
Merge branch 'main' into presleyp/destructive-dialog
presleyp Sep 19, 2022
0ac54d7
Fix users page tests
presleyp Sep 20, 2022
c787fbb
Fix workspace tests
presleyp Sep 20, 2022
b629274
Format
presleyp Sep 20, 2022
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
Next Next commit
Format
  • Loading branch information
@presleyp
presleyp committed Sep 14, 2022
commit 0dc271777930464a351338aab196ccf2e103894f
30 changes: 16 additions & 14 deletions site/src/components/Dialogs/DeleteDialog/DeleteDialog.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ import TextField from "@material-ui/core/TextField"
import Typography from "@material-ui/core/Typography"
import { Maybe } from "components/Conditionals/Maybe"
import { Stack } from "components/Stack/Stack"
import React, { useState, ChangeEvent } from "react"
import React, { ChangeEvent, useState } from "react"
import { useTranslation } from "react-i18next"
import { ConfirmDialog } from "../ConfirmDialog/ConfirmDialog"

Expand Down Expand Up @@ -51,21 +51,23 @@ export const DeleteDialog: React.FC<React.PropsWithChildren<DeleteDialogProps>>
</>
)

return <ConfirmDialog
type="delete"
hideCancel={false}
open={isOpen}
title={title}
onConfirm={onConfirm}
onClose={onCancel}
description={content}
confirmLoading={confirmLoading}
disabled={!confirmed}
/>
return (
<ConfirmDialog
type="delete"
hideCancel={false}
open={isOpen}
title={title}
onConfirm={onConfirm}
onClose={onCancel}
description={content}
confirmLoading={confirmLoading}
disabled={!confirmed}
/>
)
}

const useStyles = makeStyles((theme) => ({
warning: {
color: theme.palette.warning.light
}
color: theme.palette.warning.light,
},
}))