Skip to content

feat: mark coder_parameter as "required" #6433

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 21 commits into from
Mar 7, 2023
Merged

feat: mark coder_parameter as "required" #6433

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
d304f8b
Add required column
mtojek Mar 3, 2023
110b10b
Pass through providerd
mtojek Mar 3, 2023
8b3fe8c
Pass the required property down
mtojek Mar 3, 2023
8198eaa
Optional
mtojek Mar 6, 2023
bb1c3e2
Fix
mtojek Mar 6, 2023
93b6613
Fix
mtojek Mar 6, 2023
0813255
Fix
mtojek Mar 6, 2023
801c4df
fix
mtojek Mar 6, 2023
e804f40
CLI create: support for optional fields
mtojek Mar 6, 2023
60ffd86
Use HTML API to mark fields required
mtojek Mar 6, 2023
2c8be6c
Fix
mtojek Mar 6, 2023
5282122
Improve validation
mtojek Mar 6, 2023
659bfa2
more fixes
mtojek Mar 6, 2023
e08a3af
make fmt
mtojek Mar 6, 2023
44eca14
Fix
mtojek Mar 6, 2023
fbcbe2e
WIP
mtojek Mar 6, 2023
a688d94
Fix: test
mtojek Mar 7, 2023
16927d2
CLI update tets
mtojek Mar 7, 2023
d582472
OptionalParameterAdded
mtojek Mar 7, 2023
6e24aa3
Merge branch 'main' into 6100-required-parameter-1
mtojek Mar 7, 2023
866fef2
Fix: migration
mtojek Mar 7, 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
Next Next commit
Use HTML API to mark fields required
  • Loading branch information
@mtojek
mtojek committed Mar 6, 2023
commit 60ffd8639365c1cfa0055280dde6fd6460df15c2
1 change: 1 addition & 0 deletions site/src/components/RichParameterInput/RichParameterInput.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -169,6 +169,7 @@ const RichParameterField: React.FC<RichParameterInputProps> = ({
type={parameter.type}
size="small"
disabled={disabled}
required={parameter.required}
placeholder={parameter.default_value}
value={parameterValue}
onChange={(event) => {
Expand Down
1 change: 0 additions & 1 deletion site/src/i18n/en/workspaceBuildParametersPage.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
"title": "Workspace build parameters",
"detail": "Those values were provided by the workspace owner.",
"noParametersDefined": "This template does not use any rich parameters.",
"validationRequiredParameter": "Parameter is required.",
"validationNumberNotInRange": "Value must be between {{min}} and {{max}}.",
"validationPatternNotMatched": "{{error}} (value does not match the pattern {{pattern}}).",
"updateWorkspace": "Update workspace",
Expand Down
1 change: 0 additions & 1 deletion site/src/pages/CreateWorkspacePage/CreateWorkspacePageView.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -577,7 +577,6 @@ export const ValidationSchemaForRichParameters = (
Yup.object().shape({
name: Yup.string().required(),
value: Yup.string()
.required(t("validationRequiredParameter"))
.test("verify with template", (val, ctx) => {
const name = ctx.parent.name
const templateParameter = templateParameters.find(
Expand Down