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

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
Fix
  • Loading branch information
@mtojek
mtojek committed Mar 6, 2023
commit 44eca14fe430a22ffee25c7934fecfa3b1be966f
9 changes: 0 additions & 9 deletions codersdk/richparameters.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -82,15 +82,6 @@ func ValidateWorkspaceBuildParameter(richParameter TemplateVersionParameter, bui
return validation.Valid(richParameter.Type, value)
}

func findTemplateVersionParameter(params []TemplateVersionParameter, parameterName string) (*TemplateVersionParameter, bool) {
for _, p := range params {
if p.Name == parameterName {
return &p, true
}
}
return nil, false
}

func findBuildParameter(params []WorkspaceBuildParameter, parameterName string) (*WorkspaceBuildParameter, bool) {
if params == nil {
return nil, false
Expand Down