Skip to content
This repository was archived by the owner on Jul 20, 2023. It is now read-only.

[NOT READY] Samples demonstrating Product Search (v1p3beta1)#136

Closed
beccasaurus wants to merge 15 commits intomasterfrom
product-search-samples
Closed

[NOT READY] Samples demonstrating Product Search (v1p3beta1)#136
beccasaurus wants to merge 15 commits intomasterfrom
product-search-samples

Conversation

@beccasaurus
Copy link
Copy Markdown
Contributor

@beccasaurus beccasaurus commented Jul 24, 2018

Product Search (v1p3beta1)

This Pull Request adds the code samples for cloud.google.com documentation for Product Search How-to Guides

Product Search How-to Guides –> vision/product-search/docs/how-to

products.js

  • create_product
  • get_product
  • list_products
  • delete_product
  • update_product_labels

productSets.js

  • create_product_set
  • get_product_set
  • list_product_sets
  • delete_product_set
–> Products in productSets.js
  • add_product_to_product_set
  • list_products_in_product_set
  • remove_product_from_product_set

referenceImages.js

  • create_reference_image
  • get_reference_image
  • list_reference_images
  • delete_reference_image

similarProducts.js

  • get_similar_products_file
  • get_similar_products_uri

For the canonical versions, please see the Python Product Search Samples in python-docs-samples

These samples are in Beta and require using the v1p3beta1 API endpoint:

const vision = require('@google-cloud/vision').v1p3beta1;
const client = new vision.ProductSearchClient();

Before being ready for review & merge:

  • All samples tested & green
  • Linter passes (prettier --write *.js)
  • README scaffolded (npm run generate-scaffolding)

@ghost ghost assigned beccasaurus Jul 24, 2018
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 24, 2018
@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 24, 2018

Codecov Report

Merging #136 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #136   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           2      2           
  Lines          83     83           
=====================================
  Hits           83     83

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 253fcf2...b0ab090. Read the comment docs.

@JustinBeckwith JustinBeckwith added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jul 24, 2018
@beccasaurus
Copy link
Copy Markdown
Contributor Author

(7/24) update :: i've been busy & helping shepherd other PRs in – i will be working on wrapping this up Tuesday evening, ETA Wednesday morning 😅~ @Beccca

@beccasaurus beccasaurus mentioned this pull request Aug 9, 2018
3 tasks
@ghost ghost assigned alixhami Aug 14, 2018
@alixhami
Copy link
Copy Markdown
Contributor

I updated the region tags to the new standard tags. Update to the existing languages (Java and Python) is in progress.

@alixhami alixhami removed their assignment Aug 14, 2018
@ghost ghost assigned happyhuman Aug 22, 2018
@nirupa-kumar nirupa-kumar mentioned this pull request Sep 11, 2018
22 tasks
@beccasaurus
Copy link
Copy Markdown
Contributor Author

Moved to #192

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

Labels

cla: yes This human has signed the Contributor License Agreement. do not merge Indicates a pull request not ready for merge, due to either quality or timing.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants