Skip to content

Conversation

@munkhuushmgl
Copy link
Contributor

go/cloud-ai-2021-snippet-cleanup

Devsite update: cl/362969424

@munkhuushmgl munkhuushmgl requested a review from a team as a code owner March 17, 2021 19:06
@snippet-bot
Copy link

snippet-bot bot commented Mar 17, 2021

You are about to delete the following sample browser pages.

Here is the summary of changes.

You are about to delete 22 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

@product-auto-label product-auto-label bot added the samples Issues that are directly related to samples. label Mar 17, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Mar 17, 2021
Copy link
Contributor

@bshaffer bshaffer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks ok but I'm curious why we are completely removing all samples for Vision Product Search?

@bshaffer bshaffer merged commit 91d2d90 into GoogleCloudPlatform:master Mar 18, 2021
@rudilee
Copy link

rudilee commented Aug 29, 2021

This looks ok but I'm curious why we are completely removing all samples for Vision Product Search?

We're currently planning on using Google Cloud Vision API Product Search with PHP, so I am also curious about this matter, what does it mean by unsupported here? does the PHP client SDK doesn't support product search or something else?
Btw, the README still has examples for the product_search.php commands, this is confusing :(.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement. samples Issues that are directly related to samples.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants