From 5fa6ce8ae0c57113f25fcecbdb671a22356987d4 Mon Sep 17 00:00:00 2001 From: Peter Nemere Date: Thu, 26 Sep 2024 08:35:04 +1000 Subject: [PATCH] Raising expression group name length to 200 to temporarily fix issue with temporary RGB mixes generating names that are longer than 100 chars, will implement a different way to do RGB mixes in future --- api/ws/handlers/expression-group.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/api/ws/handlers/expression-group.go b/api/ws/handlers/expression-group.go index 3e2da4e6..1241a3fb 100644 --- a/api/ws/handlers/expression-group.go +++ b/api/ws/handlers/expression-group.go @@ -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 {