Logo is optional

Author: KillerX

backend/internal/database/migrations/00014_make_project_logo_nullable.sql

@@ -0,0 +1,7 @@
+-- +goose Up
+-- Make logo_url nullable in projects table
+ALTER TABLE projects ALTER COLUMN logo_url DROP NOT NULL;
+
+-- +goose Down
+-- Revert logo_url to NOT NULL (note: this will fail if there are NULL values)
+ALTER TABLE projects ALTER COLUMN logo_url SET NOT NULL;

backend/internal/database/queries/projects.sql

@@ -79,7 +79,7 @@ VALUES (
     @description::text,
     @startdate::timestamptz,
     @enddate::timestamptz,
-    @logourl::text,
+    sqlc.narg('logourl')::text,
     @colorprimary::text,
     @colorsecondary::text,
     @colortertiary::text,
@@ -91,10 +91,10 @@ RETURNING id, name, description, start_date, end_date, logo_url, color_primary,
 UPDATE projects
 SET
     name = COALESCE(@name::text, name),
-    description = COALESCE(@description::text, description),
+    description = COALESCE(sqlc.narg('description')::text, description),
     start_date = COALESCE(@startdate::timestamptz, start_date),
     end_date = COALESCE(@enddate::timestamptz, end_date),
-    logo_url = COALESCE(@logourl::text, logo_url),
+    logo_url = COALESCE(sqlc.narg('logourl')::text, logo_url),
     color_primary = COALESCE(@colorprimary::text, color_primary),
     color_secondary = COALESCE(@colorsecondary::text, color_secondary),
     color_tertiary = COALESCE(@colortertiary::text, color_tertiary),

backend/internal/database/sqlc/models.go

@@ -43,14 +43,19 @@ func projectByIDBatchFunc(db *database.DB, c *cache.CacheWithRegistry) func(cont
 
 			// Convert to GraphQL model and populate cache
 			for _, row := range rows {
+				var logo *string
+				if row.LogoUrl != nil {
+					logo = row.LogoUrl
+				}
+
 				project := &model.Project{
 					ID:          row.ID,
 					Name:        row.Name,
 					Description: row.Description,
 					StartDate:   scalars.DateTime{Time: row.StartDate.Time},
 					EndDate:     scalars.DateTime{Time: row.EndDate.Time},
 					Branding: &model.Branding{
-						Logo: row.LogoUrl,
+						Logo: logo,
 						Colors: &model.Colors{
 							Primary:   row.ColorPrimary,
 							Secondary: row.ColorSecondary,

backend/internal/database/sqlc/projects.sql.go

@@ -41,14 +41,19 @@ func projectsByUserBatchFunc(db *database.DB, c *cache.CacheWithRegistry) func(c
 
 			// Group projects by user ID and convert to GraphQL model
 			for _, row := range rows {
+				var logo *string
+				if row.LogoUrl != nil {
+					logo = row.LogoUrl
+				}
+
 				project := &model.Project{
 					ID:          row.ID,
 					Name:        row.Name,
 					Description: row.Description,
 					StartDate:   scalars.DateTime{Time: row.StartDate.Time},
 					EndDate:     scalars.DateTime{Time: row.EndDate.Time},
 					Branding: &model.Branding{
-						Logo: row.LogoUrl,
+						Logo: logo,
 						Colors: &model.Colors{
 							Primary:   row.ColorPrimary,
 							Secondary: row.ColorSecondary,

backend/internal/graph/api/generated.go

@@ -76,7 +76,7 @@ type Track {
 }
 
 type Branding {
-    logo: String!
+    logo: String
     colors: Colors!
     rounding: Int! # border-radius in pixels
 }
@@ -326,7 +326,7 @@ input AgeRangeInput {
 }
 
 input BrandingInput {
-    logo: String!
+    logo: String
     colors: ColorsInput!
     rounding: Int!
 }