- removed not used call of sval.String() from migrateEmailTemplates

- Added pointer as second argument of Unmarshal call in MarshalYAML

pkg/envoy/yaml/compose_module_marshal.go

@@ -97,7 +97,9 @@ func (n *composeModule) Encode(ctx context.Context, doc *Document, state *envoy.
 func (c *composeModule) MarshalYAML() (interface{}, error) {
 	// Get a struct from the raw JSON string
 	auxMeta := make(map[string]interface{})
-	json.Unmarshal(c.res.Meta, auxMeta)
+	if err := json.Unmarshal(c.res.Meta, &auxMeta); err != nil {
+		return nil, err
+	}
 
 	nn, err := makeMap(
 		"handle", c.res.Handle,

pkg/provision/migrations_202103_templates.go

@@ -81,7 +81,6 @@ func migrateEmailTemplates(ctx context.Context, log *zap.Logger, s store.Storer)
 				continue
 			}
 
-			sval.String()
 			tmpl.ID = id.Next()
 			tmpl.CreatedAt = time.Now()
 			tmpl.Template = sval.String()