-
Notifications
You must be signed in to change notification settings - Fork 7
Conversation
5d27205
to
dba9727
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
conditionally approved waiting for @eefahy on question in comment
@@ -58,6 +59,7 @@ func (d *depositResource) Handle(params operations.DepositResourceParams, agent | |||
WithID(uuid). | |||
WithExternalIdentifier(externalID). | |||
WithVersion(1) | |||
(*resource.Administrative())["created"] = time.Now().UTC().Format(time.RFC3339) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@eefahy did you / we ever move the ball on capturing or using the dynamodb datetime created for the sake of this admin field? i think you said it was captured, it was just a question of putting it in the metadata or resource responses for sake of clients + users.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
never mind, dynamodb date is unreliable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
waiting to run local tests after rebase.
note this doesn't add admin metadata for File metadata. |
Fixes #408