rawlinp commented on a change in pull request #5922:
URL: https://github.com/apache/trafficcontrol/pull/5922#discussion_r652122130
##########
File path: lib/go-tc/deliveryservices.go
##########
@@ -192,18 +194,416 @@ type DeliveryServiceFieldsV31 struct {
// DeliveryServiceV40 is a Delivery Service as it appears in version 4.0 of the
// Traffic Ops API.
type DeliveryServiceV40 struct {
Review comment:
Ok, there might be another option. If you add a new `LastUpdated
time.Time` field with the same JSON tags (and probably db tags for good
measure) to `DeliveryServiceV40`, when doing JSON marshalling at V40, it will
take precedence over the _embedded_ `LastUpdated`. We just have to make sure to
set both fields before marshalling the response.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
