-
Notifications
You must be signed in to change notification settings - Fork 2k
feat: add storage batch operations samples #4180
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
base: main
Are you sure you want to change the base?
feat: add storage batch operations samples #4180
Conversation
e7476df
to
c1d08bd
Compare
Here is the summary of changes. You are about to add 6 region tags.
This comment is generated by snippet-bot.
|
c1d08bd
to
b424e2c
Compare
@ddelgrosso1 Just a friendly reminder to review this PR when you get a chance. Thanks! |
@ddelgrosso1 Could you please merge this PR if possible? |
…on & Error Handling) Improve the robustness and clarity of the Storage Batch Operations API samples: * **Error Handling:** Wraps all asynchronous Batch Operations API samples (Create, Get, List, Cancel) in `try...catch` blocks for production readiness. * **Specific Diagnostics:** Adds specific gRPC error code checks (`NOT_FOUND`, `FAILED_PRECONDITION`) within the `catch` blocks to provide detailed diagnostic feedback to users regarding job state or non-existence. * **Documentation:** Clarifies JSDoc for all function parameters (e.g., `projectId`, `jobId`, `objectPrefix`), ensuring examples and types are clear for developers.
fb51efe
to
6f5d870
Compare
Description
Fixes #
Note: Before submitting a pull request, please open an issue for discussion if you are not associated with Google.
Checklist
npm test
(see Testing)npm run lint
(see Style)GoogleCloudPlatform/nodejs-docs-samples
. Not a fork.