Skip to content
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

Add server_domain field in questions requests to Robotoff #2794

Closed
raphael0202 opened this issue Dec 18, 2019 · 0 comments
Closed

Add server_domain field in questions requests to Robotoff #2794

raphael0202 opened this issue Dec 18, 2019 · 0 comments
Labels
🐛 bug This is a bug, not a feature request. 🤖 Robotoff

Comments

@raphael0202
Copy link
Contributor

raphael0202 commented Dec 18, 2019

We want Robotoff insights to be specific to each website (off, off-pro, openbeautyfacts, openpetfoodfacts, openproductfacts) and to each environment (dev, prod).
This has been taken care of on Robotoff side (see openfoodfacts/robotoff#47), we now need to only display insights that match the server domain (website + environment).
A server_domain field is currently sent to the /api/v1/webhook/product endpoint. We also need to send this field when requesting new Robotoff questions on /api/v1/questions/{barcode}. See the API documentation for more information.

Known values for the server_domain field (according what robotoff webhook endpoint receives):

api.pro.openfoodfacts.org
api.openfoodfacts.org
api.openfoodfacts.net
api.pro.openfoodfacts.dev

For completeness, here is the issue that added Robotoff webhook ping: #1707.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug This is a bug, not a feature request. 🤖 Robotoff
Projects
Status: Done
Development

No branches or pull requests

1 participant