Skip to content

Template settings fixes/kira pilot #3668

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 13 commits into from
Aug 24, 2022
Merged

Template settings fixes/kira pilot #3668

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
cf85bfe
using hours instead of seconds
Kira-Pilot Aug 23, 2022
8b8882f
checking out
Kira-Pilot Aug 24, 2022
7b5a78c
Merge remote-tracking branch 'origin/main' into template-settings-fix…
Kira-Pilot Aug 24, 2022
45a41d1
added ttl tests
Kira-Pilot Aug 24, 2022
3bab482
added description validation and tests
Kira-Pilot Aug 24, 2022
94abca9
added some helper text
Kira-Pilot Aug 24, 2022
b7ce020
fix typing
Kira-Pilot Aug 24, 2022
1eda40c
Update site/src/pages/TemplateSettingsPage/TemplateSettingsForm.tsx
Kira-Pilot Aug 24, 2022
67b65d0
Merge remote-tracking branch 'origin/main' into template-settings-fix…
Kira-Pilot Aug 24, 2022
ac67ccb
ran prettier
Kira-Pilot Aug 24, 2022
5f74e95
added ttl of 0 test
Kira-Pilot Aug 24, 2022
db10782
typo
Kira-Pilot Aug 24, 2022
15e1ad2
PR feedback
Kira-Pilot Aug 24, 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
added some helper text
  • Loading branch information
@Kira-Pilot
Kira-Pilot committed Aug 24, 2022
commit 94abca98b79ebfc11dca37f45baf32db2562e88e
10 changes: 10 additions & 0 deletions site/src/pages/TemplateSettingsPage/TemplateSettingsForm.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ import InputAdornment from "@material-ui/core/InputAdornment"
import Popover from "@material-ui/core/Popover"
import { makeStyles } from "@material-ui/core/styles"
import TextField from "@material-ui/core/TextField"
import Typography from "@material-ui/core/Typography"
import { Template, UpdateTemplateMeta } from "api/typesGenerated"
import { OpenDropdown } from "components/DropdownArrows/DropdownArrows"
import { FormFooter } from "components/FormFooter/FormFooter"
Expand All @@ -24,6 +25,10 @@ export const Language = {
selectEmoji: "Select emoji",
ttlMaxError: "Please enter a limit that is less than or equal to 168 hours (7 days).",
descriptionMaxError: "Please enter a description that is less than or equal to 128 characters.",
ttlHelperText: (
ttl?: number,
): string => `Workspaces created from this template will, by default, shut down ${ttl}
hours after starting.`,
}

const MAX_DESCRIPTION_CHAR_LIMIT = 128
Expand Down Expand Up @@ -168,6 +173,11 @@ export const TemplateSettingsForm: FC<TemplateSettingsForm> = ({
variant="outlined"
type="number"
/>
{!form.errors.max_ttl_ms && (
<Typography variant="subtitle2">
{Language.ttlHelperText(form.values.max_ttl_ms)}
</Typography>
)}
</Stack>

<FormFooter onCancel={onCancel} isLoading={isSubmitting} />
Expand Down