Repository navigation
Add a new_batch_http_request() method to services - #105
Conversation
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
|
Spelling errors on both the first and last lines of your commit message, right? Plus an erroneous second space after a period in the middle paragraph (http://www.slate.com/articles/technology/technology/2011/01/space_invaders.html). |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
|
Given the dynamism here I'll want @craigcitro to be looking at this as well as myself. |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This method can be used to get a BatchHttpRequest object with the batch_uri calculated from the rootUrl and batchPath fields from the discovery doc. Without this change, users have to know what the batch url for newer api endpoints is. For example: batch = BatchHttpRequest(batch_uri=http://zoo.googleapis.com/batch) With this change, the syntax is easier: zoo = build('zoo', 'v1', http=http) batch = zoo.new_batch_http_request()
|
Took me a little while to figure out git rebase, but this now looks like a single commit like we wanted, right? |
|
LGTM. Any urgent need for this to be merged right away? |
|
LGTM |
Add a new_batch_http_request(callback=None) method to services.
This method can be used to get a BatchHttpRequest object with the
batch_uri calculated from the rootUrl and batchPath fields from the
discovery doc.
Without this change, users have to know what the batch url for newer api
endpoints is. For example:
batch = BatchHttpRequest(batch_uri=http://zoo.googleapis.com/batch)
With this change, the syntax is easier:
zoo = build('zoo', 'v1', http=http)
batch = zoo.new_batch_http_request()