-
-
Notifications
You must be signed in to change notification settings - Fork 383
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
Omega 6 and 9 placement in table #8395
Comments
It might be linked to %nutriments_tables in Food.pm It seems to use dashes ("-", "--") to set a kind of hierarchy, but I don't know if 3 level hierarchy is possible, "---" as omega-3 and omega-6 seems to have sub items.) |
Can you plz elaborate |
Hi @Tanujgarg37 sorry for late reply. But I think you may have to deep dive into the javascript that builds the table. |
Sure I will look into it
…On Thu, Jul 27, 2023, 8:12 PM Alex Garel ***@***.***> wrote:
Hi @Tanujgarg37 <https://github.com/Tanujgarg37> sorry for late reply.
But I think you may have to deep dive into the javascript that builds the
table.
I think the %nutriments_tables in Food.pm is the one that drives the way
nutriments are presented. I added a comment not so long ago, it might help
you.
See
https://github.com/openfoodfacts/openfoodfacts-server/blob/main/lib/ProductOpener/Food.pm#L336
—
Reply to this email directly, view it on GitHub
<#8395 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AU5CCEKTRGMKK6LKQPFFXKLXSJ433ANCNFSM6AAAAAAXVQBHJU>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Assign this to me. i am already working on another issue related the to the same table, i have a good understanding of the table. can you please assign the issue to me? |
Sorry I will do it by myself
…On Sun, Aug 13, 2023, 10:34 PM owais fayaz ***@***.***> wrote:
Assign this to me. i am already working on another issue related the to
the same table, i have a good understanding of the table. can you please
assign the issue to me?
—
Reply to this email directly, view it on GitHub
<#8395 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AU5CCEPB2GVEAHP4LIETDE3XVECK3ANCNFSM6AAAAAAXVQBHJU>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
This issue has been open 90 days with no activity. Can you give it a little love by linking it to a parent issue, adding relevant labels and projets, creating a mockup if applicable, adding code pointers from https://github.com/openfoodfacts/openfoodfacts-server/blob/main/.github/labeler.yml, giving it a priority, editing the original issue to have a more comprehensive description… Thank you very much for your contribution to 🍊 Open Food Facts |
see #9569 |
Describe the bug
The fats Omega 6 and 9 appear now as a separate row, the could be place under other fats.
To Reproduce
See this product: https://world.openfoodfacts.org/product/3329489000309/huile-d-olive-bio-vierge-extra-100-espagne-1l-grand-olio
Expected behavior
Put Omega 6 under polyunsaturated fats
Put Omega 9 under monounsaturated dats
Screenshots
No response
Additional context
No response
Type of device
Browser
Browser version
No response
Number of products impacted
No response
Time per product
No response
The text was updated successfully, but these errors were encountered: