Skip to content

Commit

Permalink
Merge pull request #324 from pixlise/bugfix/expr-group-name-length
Browse files Browse the repository at this point in the history
Raising expression group name length to 200 to temporarily fix issue …
  • Loading branch information
pnemere authored Sep 25, 2024
2 parents 100fc35 + 5fa6ce8 commit ec93460
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion api/ws/handlers/expression-group.go
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,9 @@ func HandleExpressionGroupGetReq(req *protos.ExpressionGroupGetReq, hctx wsHelpe
}

func validateExpressionGroup(egroup *protos.ExpressionGroup) error {
if err := wsHelpers.CheckStringField(&egroup.Name, "Name", 1, 100); err != nil {
// NOTE: name allows for 200 chars. Original was 100. Raised due to temporary RGB mixes generating names that are too long. We intend
// to fix RGB mixes in a different way in future and should lower this back at that point
if err := wsHelpers.CheckStringField(&egroup.Name, "Name", 1, 200); err != nil {
return err
}
if err := wsHelpers.CheckStringField(&egroup.Description, "Description", 0, wsHelpers.DescriptionFieldMaxLength); err != nil {
Expand Down

0 comments on commit ec93460

Please sign in to comment.