-
-
Notifications
You must be signed in to change notification settings - Fork 282
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
Hide the search env switcher by default #5855
Comments
That goes beyond hiding those radio buttons: barcode search are performed on all OxF. |
We will keep barcode search because @stephanegigandet will provide a unified API next week. |
The side-effect may be to consider all products as food, cf. |
Mmmm, I've been thinking about your comment for 2 days. What could go wrong? |
If we search for food by default, product_type will be food ? |
Whatever we find on the food server will be considered food in Smoothie. Same for beauty, ... I had the impression that @stephanegigandet's work ("unified search API") would make it possible to look for a barcode on the food server and to retrieve potentially a beauty product. If so it will be considered as food in Smoothie. Next steps would be anyway:
|
Yes, a universal API for scanning (by barcode) will be available this week. The same goes for product addition, where we can add a product on OxF. However, here for the search env switcher, the idea is that we can only search for food (just like if "Food" was selected for all queries). I know we can have some edge cases, where the full-text search wouldn't find a beauty product… But, once again, we already have the issue with the switcher. |
Then I did understand well and there are 2 issues to fix. There's no ProductField called "product_type" - we don't retrieve that info from the server, which definitely forces Smoothie to consider all products from the food server to be food. In Smoothie, we must now retrieve that new product field, and not overwrite that field provided by the server with the "natural" product type related to the server (in our case, "food"). |
👌 |
Ah ok, I thought we auto added product_type based on the pressed button, or the server base url as a stopgap measure. |
For barcode search we loop on all 4 servers, stopping the first time we find something, and set the related server product type. |
Hi everyone!
For the next release of the app, we still want to keep some parts of the OxF integration (with the scanner for example), but we're not ready to promote it yet.
That's why we want to hide the OxF switcher from search:
However, hiding doesn't mean removing it all.
In the preferences, the idea is to add a switch to hide/show the swicher (default value: invisible).
The text was updated successfully, but these errors were encountered: