nodejs-docs-samples icon indicating copy to clipboard operation
nodejs-docs-samples copied to clipboard

feat: add storage batch operations samples

Open thiyaguk09 opened this issue 3 months ago • 6 comments

Description

Fixes #<ISSUE-NUMBER>

Note: Before submitting a pull request, please open an issue for discussion if you are not associated with Google.

Checklist

  • [ ] I have followed guidelines from CONTRIBUTING.MD and Samples Style Guide
  • [ ] Tests pass: npm test (see Testing)
  • [ ] Lint pass: npm run lint (see Style)
  • [ ] Required CI tests pass (see CI testing)
  • [ ] These samples need a new API enabled in testing projects to pass (let us know which ones)
  • [ ] These samples need a new/updated env vars in testing projects set to pass (let us know which ones)
  • [ ] This pull request is from a branch created directly off of GoogleCloudPlatform/nodejs-docs-samples. Not a fork.
  • [ ] This sample adds a new sample directory, and I updated the CODEOWNERS file with the codeowners for this sample
  • [ ] This sample adds a new sample directory, and I created GitHub Actions workflow for this sample
  • [ ] This sample adds a new Product API, and I updated the Blunderbuss issue/PR auto-assigner with the codeowners for this sample
  • [ ] Please merge this PR for me once it is approved

Note: Any check with (dev), (experimental), or (legacy) can be ignored and should not block your PR from merging (see CI testing).

thiyaguk09 avatar Sep 11 '25 06:09 thiyaguk09

Here is the summary of changes.

You are about to add 6 region tags.

This comment is generated by snippet-bot. If you find problems with this result, please file an issue at: https://github.com/googleapis/repo-automation-bots/issues. To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • [ ] Refresh this comment

snippet-bot[bot] avatar Sep 12 '25 09:09 snippet-bot[bot]

@ddelgrosso1 Just a friendly reminder to review this PR when you get a chance. Thanks!

thiyaguk09 avatar Sep 22 '25 08:09 thiyaguk09

@ddelgrosso1 Could you please merge this PR if possible?

thiyaguk09 avatar Sep 30 '25 08:09 thiyaguk09

Looks like Storage Intelligence is not enabled for the long-door-651 project. You will need to reach out to a member of the Cloud SDK Node team to get it enabled.

ddelgrosso1 avatar Oct 27 '25 19:10 ddelgrosso1

@ddelgrosso1 Storage Intelligence permission has been enabled, and I am currently unable to run tests from my end. Could you please review it once?

thiyaguk09 avatar Oct 30 '25 07:10 thiyaguk09

@ddelgrosso1, @chandra-siri Could you please run the test cases again? I don't have permission to do so on my end. I have received confirmation that Storage Intelligence permission has been granted.

thiyaguk09 avatar Nov 18 '25 05:11 thiyaguk09