Skip to content

Allowing dict (as an Entity) for property values.#3927

Merged
dhermes merged 2 commits intogoogleapis:masterfrom
dhermes:fix-3923
Sep 8, 2017
Merged

Allowing dict (as an Entity) for property values.#3927
dhermes merged 2 commits intogoogleapis:masterfrom
dhermes:fix-3923

Conversation

@dhermes
Copy link
Contributor

@dhermes dhermes commented Sep 6, 2017

Fixes #3923.

@dhermes dhermes added api: datastore Issues related to the Datastore API. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. labels Sep 6, 2017
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 6, 2017
@dhermes
Copy link
Contributor Author

dhermes commented Sep 6, 2017

Man, Python 2 I am ready for you to disappear: https://circleci.com/gh/GoogleCloudPlatform/google-cloud-python/3177

@dhermes dhermes closed this Sep 6, 2017
@dhermes dhermes reopened this Sep 6, 2017
@dhermes
Copy link
Contributor Author

dhermes commented Sep 7, 2017

@tseaver thanks for the review. @lukesneeringer Can you also take a look? I'm a bit worried I've over-simplified this fix.

@dhermes dhermes merged commit ed906ee into googleapis:master Sep 8, 2017
@dhermes dhermes deleted the fix-3923 branch September 8, 2017 15:47
parthea pushed a commit that referenced this pull request Nov 24, 2025
* Allowing `dict` (as an `Entity`) for property values.

Fixes #3923.

* Using explicit unicode / bytes in new dict->Entity unit tests.
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. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants