chore: make telemetry use_classic_parameter_flow nullable (#18547)

Emyrk · web-flow · commit 06c997a1003b · 2025-06-24T13:38:12.000-05:00
diff --git a/coderd/telemetry/telemetry.go b/coderd/telemetry/telemetry.go
@@ -33,6 +33,7 @@ import (
 	clitelemetry "github.com/coder/coder/v2/cli/telemetry"
 	"github.com/coder/coder/v2/coderd/database"
 	"github.com/coder/coder/v2/coderd/database/dbtime"
+	"github.com/coder/coder/v2/coderd/util/ptr"
 	"github.com/coder/coder/v2/codersdk"
 	tailnetproto "github.com/coder/coder/v2/tailnet/proto"
 )
@@ -1090,7 +1091,7 @@ func ConvertTemplate(dbTemplate database.Template) Template {
 		AutostartAllowedDays:          codersdk.BitmapToWeekdays(dbTemplate.AutostartAllowedDays()),
 		RequireActiveVersion:          dbTemplate.RequireActiveVersion,
 		Deprecated:                    dbTemplate.Deprecated != "",
-		UseClassicParameterFlow:       dbTemplate.UseClassicParameterFlow,
+		UseClassicParameterFlow:       ptr.Ref(dbTemplate.UseClassicParameterFlow),
 	}
 }
 
@@ -1397,7 +1398,7 @@ type Template struct {
 	AutostartAllowedDays           []string `json:"autostart_allowed_days"`
 	RequireActiveVersion           bool     `json:"require_active_version"`
 	Deprecated                     bool     `json:"deprecated"`
-	UseClassicParameterFlow        bool     `json:"use_classic_parameter_flow"`
+	UseClassicParameterFlow        *bool    `json:"use_classic_parameter_flow"`
 }
 
 type TemplateVersion struct {

Original file line number	Diff line number	Diff line change
`@@ -33,6 +33,7 @@ import (`
`33`	`33`	`clitelemetry "github.com/coder/coder/v2/cli/telemetry"`
`34`	`34`	`"github.com/coder/coder/v2/coderd/database"`
`35`	`35`	`"github.com/coder/coder/v2/coderd/database/dbtime"`
	`36`	`+ "github.com/coder/coder/v2/coderd/util/ptr"`
`36`	`37`	`"github.com/coder/coder/v2/codersdk"`
`37`	`38`	`tailnetproto "github.com/coder/coder/v2/tailnet/proto"`
`38`	`39`	`)`
`@@ -1090,7 +1091,7 @@ func ConvertTemplate(dbTemplate database.Template) Template {`
`1090`	`1091`	`AutostartAllowedDays: codersdk.BitmapToWeekdays(dbTemplate.AutostartAllowedDays()),`
`1091`	`1092`	`RequireActiveVersion: dbTemplate.RequireActiveVersion,`
`1092`	`1093`	`Deprecated: dbTemplate.Deprecated != "",`
`1093`		`- UseClassicParameterFlow: dbTemplate.UseClassicParameterFlow,`
	`1094`	`+ UseClassicParameterFlow: ptr.Ref(dbTemplate.UseClassicParameterFlow),`
`1094`	`1095`	`}`
`1095`	`1096`	`}`
`1096`	`1097`
`@@ -1397,7 +1398,7 @@ type Template struct {`
`1397`	`1398`	AutostartAllowedDays []string `json:"autostart_allowed_days"`
`1398`	`1399`	RequireActiveVersion bool `json:"require_active_version"`
`1399`	`1400`	Deprecated bool `json:"deprecated"`
`1400`		- UseClassicParameterFlow bool `json:"use_classic_parameter_flow"`
	`1401`	+ UseClassicParameterFlow *bool `json:"use_classic_parameter_flow"`
`1401`	`1402`	`}`
`1402`	`1403`
`1403`	`1404`	`type TemplateVersion struct {`