Skip to content

Conversation

@mziccard
Copy link
Contributor

This PR fixes #732 on our side.

Since wrongly-formattted timestamps might still be around as part of older deprecated objects we must still save timestamps as strings and expose a string getter. This PR however hides the string setter and repaces todos with explanatory comments.

@mziccard mziccard added the api: compute Issues related to the Compute Engine API. label Jul 22, 2016
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 22, 2016
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 85.865% when pulling 41036ec on mziccard:fix-image-timestamp into 9549dfc on GoogleCloudPlatform:master.

@mziccard
Copy link
Contributor Author

mziccard commented Aug 8, 2016

/cc @lesv

@lesv
Copy link
Contributor

lesv commented Aug 8, 2016

LGTM

@mziccard mziccard merged commit e1309f0 into googleapis:master Aug 9, 2016
meltsufin pushed a commit that referenced this pull request Dec 22, 2025
🤖 I have created a release *beep* *boop*
---


### Updating meta-information for bleeding-edge SNAPSHOT release.

---
This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: compute Issues related to the Compute Engine API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Image's deprecationStatus can hold wrongly formatted timestamps

4 participants