Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding helpers for interacting with properties in Entity protobuf. #1340

Merged
merged 1 commit into from
Jan 7, 2016

Conversation

dhermes
Copy link
Contributor

@dhermes dhermes commented Jan 3, 2016

This is because the repeated property message field in Entity becomes a properties map field in v1beta3. This makes adding new properties and iterating through all properties very different, so we add a helper to ease the transition from v1beta2 to v1beta3.

This is because the repeated `property` message field in `Entity`
becomes a `properties` map field in `v1beta3`. This makes adding
new properties and iterating through all properties very
different, so we add a helper to ease the transition from
`v1beta2` to `v1beta3`.
@dhermes dhermes added the api: datastore Issues related to the Datastore API. label Jan 3, 2016
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jan 3, 2016
@dhermes dhermes mentioned this pull request Jan 3, 2016
49 tasks

:rtype: :class:`gcloud.datastore._generated.entity_pb2.Value`
:returns: The new ``Value`` protobuf that was added to the entity.
"""

This comment was marked as spam.

This comment was marked as spam.

@tseaver
Copy link
Contributor

tseaver commented Jan 7, 2016

LGTM

dhermes added a commit that referenced this pull request Jan 7, 2016
Adding helpers for interacting with properties in Entity protobuf.
@dhermes dhermes merged commit ee25a8d into googleapis:master Jan 7, 2016
@dhermes dhermes deleted the v1beta3-entity-prop-access branch January 7, 2016 22:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: datastore Issues related to the Datastore API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants