Commit1267c9c

authored

fix: ensure reason present for workspace autoupdated notification (#17935)

Fixes#17930Update the `WorkspaceAutoUpdated` notification to only display thereason if it is present.

1 parent769c9ee commit1267c9cCopy full SHA for 1267c9c

File tree

-1

lines changed

-1

lines changed

Lines changed: 6 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -349,13 +349,18 @@ func (e *Executor) runOnce(t time.Time) Stats {`
`349`	`349`	`nextBuildReason=string(nextBuild.Reason)`
`350`	`350`	`}`
`351`	`351`
	`352`	`+templateVersionMessage:=activeTemplateVersion.Message`
	`353`	`+iftemplateVersionMessage=="" {`
	`354`	`+templateVersionMessage="None provided"`
	`355`	`+}`
	`356`	`+`
`352`	`357`	`if_,err:=e.notificationsEnqueuer.Enqueue(e.ctx,ws.OwnerID,notifications.TemplateWorkspaceAutoUpdated,`
`353`	`358`	`map[string]string{`
`354`	`359`	`"name":ws.Name,`
`355`	`360`	`"initiator":"autobuild",`
`356`	`361`	`"reason":nextBuildReason,`
`357`	`362`	`"template_version_name":activeTemplateVersion.Name,`
`358`		`-"template_version_message":activeTemplateVersion.Message,`
	`363`	`+"template_version_message":templateVersionMessage,`
`359`	`364`	`},"autobuild",`
`360`	`365`	`// Associate this notification with all the related entities.`
`361`	`366`	`ws.ID,ws.OwnerID,ws.TemplateID,ws.OrganizationID,`

Comments

(0)