jwetzell/showbridge-go
1
0
Fork 0
mirror of https://github.com/jwetzell/showbridge-go.git synced 2026-08-31 10:48:59 +00:00
Code Issues Packages Projects Releases 10 Wiki Activity

Merge pull request #166 from jwetzell/fix/params-schema-required

Browse Source 
don't make params schema required if all properties are optional
This commit is contained in:
Joel Wetzell
2026-05-20 17:30:28 -05:00
committed by GitHub
parent 70c4aa172a 1bbe0cc045
commit fa4eeddbbf
2 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
internal/schema/modules.go
+3 -1
View File
@@ -43,7 +43,9 @@ func GetModulesSchema() *jsonschema.Schema {
} }
if mod.ParamsSchema != nil { if mod.ParamsSchema != nil {
moduleSchema.Properties["params"] = mod.ParamsSchema moduleSchema.Properties["params"] = mod.ParamsSchema
moduleSchema.Required = append(moduleSchema.Required, "params") if len(mod.ParamsSchema.Required) > 0 {
moduleSchema.Required = append(moduleSchema.Required, "params")
}
} }
moduleDefinitionSchemas = append(moduleDefinitionSchemas, moduleSchema) moduleDefinitionSchemas = append(moduleDefinitionSchemas, moduleSchema)
} }
internal/schema/processors.go
+3 -1
View File
@@ -39,7 +39,9 @@ func GetProcessorsSchema() *jsonschema.Schema {
} }
if proc.ParamsSchema != nil { if proc.ParamsSchema != nil {
processorSchema.Properties["params"] = proc.ParamsSchema processorSchema.Properties["params"] = proc.ParamsSchema
processorSchema.Required = append(processorSchema.Required, "params") if len(proc.ParamsSchema.Required) > 0 {
processorSchema.Required = append(processorSchema.Required, "params")
}
} }
processorDefinitionSchemas = append(processorDefinitionSchemas, processorSchema) processorDefinitionSchemas = append(processorDefinitionSchemas, processorSchema)
} }