-
Notifications
You must be signed in to change notification settings - Fork 643
introduce Translate API #1023
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
introduce Translate API #1023
Conversation
0d3ac92 to
12d1d4a
Compare
|
@callmehiphop PTAQL. This one's pretty small. I just want to be sure I'm not doing anything too crazy before locking it up with the unit tests. |
lib/translate/index.js
Outdated
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 comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
|
@stephenplusplus LGTM :) unit test away! |
|
Thanks for the review! |
README.md
Outdated
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.
|
/cc @ilsung Does Translate work with service account credentials (ie, keyfile.json) ? |
|
Tests added: stephenplusplus@1ebcde0 |
1ebcde0 to
a72427a
Compare
|
Marking as blocked until we hear from @ilsung about authenticating with a service account keyfile. (Thanks in advance for your help!) |
a72427a to
027a4ba
Compare
|
Sorry for the delay -- No, the Translate API does not support service account creds since it doesn't use OAuth. A key is required. |
|
Thanks for clearing that up. @callmehiphop if this LGTY, merge away. |
|
🍰 |
🤖 I have created a release \*beep\* \*boop\* --- ## [2.15.0](https://github.com/googleapis/gax-nodejs/compare/v2.14.1...v2.15.0) (2021-06-10) ### Features * expose cert and key for mTLS ([#1014](https://github.com/googleapis/gax-nodejs/issues/1014)) ([f025fc1](https://github.com/googleapis/gax-nodejs/commit/f025fc155c505792bc74929a7803a2df63c331cd)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
* feat: add jobs.delete * fix test * refactor * lint * lint * update jobs test * refactor * lint * remove unnecessary DeleteCallback * remove unnecessary DeleteCallback * update * add deleteJobs() to system-test to delete stale job resources * remove unnecessary jobs cleanup
🤖 I have created a release \*beep\* \*boop\* --- ## [5.10.0](https://github.com/googleapis/nodejs-bigquery/compare/v5.9.3...v5.10.0) (2022-01-10) ### Features * Add jobs.delete and update to latest discovery types ([#1023](https://github.com/googleapis/nodejs-bigquery/issues/1023)) ([12f7771](https://github.com/googleapis/nodejs-bigquery/commit/12f77710accfca8c512d021a1c75a732c6c666f7)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Fixes #977
To Dos
Preview