-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Bring back .withDefaultParam #278
Comments
Maybe this could be make more predictable by making those params immutable, for example they could be passed to the client initializer function. |
Yes this the only place I would condone putting the change. What are your thoughts on something like the following: var google = require('googleapis');
var datastore = google.datastore({
version: 'v1beta2',
defaultParams: { datasetId: 'abc123' },
auth: auth
}); |
Or instead of |
Maybe just |
Yes, |
@proppy found the feature useful. Used to set default params for every request. I always thought it was more unpredictable so it was never transitioned to the new library.
See discussion: https://github.com/GoogleCloudPlatform/google-cloud-datastore/pull/47/files#r17020602
The text was updated successfully, but these errors were encountered: