Skip to content

Conversation

@stephenplusplus
Copy link
Contributor

Fixes #910

The bug originally caught here by @electricFeel (thanks!) resulted in a re-factor, also bringing in some of the sane changes from #897 (the datastore v1beta3 upgrade).

PTAL asap, as we should get out a patch release as soon as possible.

@stephenplusplus stephenplusplus added type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. api: datastore Issues related to the Datastore API. labels Oct 14, 2015
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 14, 2015
@stephenplusplus stephenplusplus mentioned this pull request Oct 14, 2015
19 tasks

This comment was marked as spam.

This comment was marked as spam.

@stephenplusplus stephenplusplus force-pushed the spp--datastore-response-parsing branch from 31615c5 to 8b2bbf2 Compare October 14, 2015 17:45
callmehiphop added a commit that referenced this pull request Oct 14, 2015
…parsing

datastore: properly parse HTTP responses from the API
@callmehiphop callmehiphop merged commit e2371f5 into googleapis:master Oct 14, 2015
@callmehiphop
Copy link
Contributor

Thank you, sir!

miguelvelezsa pushed a commit that referenced this pull request Jul 23, 2025
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
GautamSharda pushed a commit that referenced this pull request Jan 14, 2026
Source-Link: googleapis/synthtool@7751597
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-nodejs:latest@sha256:a1e45c1b53fceb5fa7fd5c49cf898048c210eb07001a3355548e7a4545200ee8

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
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: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants