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
fix typing
  • Loading branch information
@Kira-Pilot
Kira-Pilot committed Aug 24, 2022
commit b7ce0202ca1fc0c62195af7085958552cf5f6502
4 changes: 2 additions & 2 deletions site/src/pages/TemplateSettingsPage/TemplateSettingsForm.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ export const Language = {
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,
ttl: number,
): string => `Workspaces created from this template will, by default, shut down ${ttl}
hours after starting.`,
}
Expand Down Expand Up @@ -173,7 +173,7 @@ export const TemplateSettingsForm: FC<TemplateSettingsForm> = ({
variant="outlined"
type="number"
/>
{!form.errors.max_ttl_ms && (
{!form.errors.max_ttl_ms && form.values.max_ttl_ms && (
<Typography variant="subtitle2">
{Language.ttlHelperText(form.values.max_ttl_ms)}
</Typography>
Expand Down