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

Display app_name and app_uuid in edit history #9731

Closed
stephanegigandet opened this issue Feb 2, 2024 · 4 comments · Fixed by #9978
Closed

Display app_name and app_uuid in edit history #9731

stephanegigandet opened this issue Feb 2, 2024 · 4 comments · Fixed by #9978
Labels
good first issue Welcome to Open Food Facts. This issue should be approachable if you're new. Get in touch for help. product history We have kept 10 years of product revisions. This is useful to monitor edits & product improvements

Comments

@stephanegigandet
Copy link
Contributor

From Slack:

David Pryor
il y a 12 heures
@stephane

https://world.openfoodfacts.org/product/01782877499/jack-links-beef-jerky-35g-teriyaki-jack-link-s
I submitted with my personal account and a uuid of
WjU4TEtJa1B2T2hWdWZNNzhnL3YyZUJvNXE2TUIxR05LK2crSVE9PQ

--

The uuid was saved in changes.sto but is not displayed.

@alexgarel alexgarel added the good first issue Welcome to Open Food Facts. This issue should be approachable if you're new. Get in touch for help. label Feb 7, 2024
@teolemon teolemon added the product history We have kept 10 years of product revisions. This is useful to monitor edits & product improvements label Feb 8, 2024
@deveshidwivedi
Copy link
Contributor

Hi! I wish to work on this. I understand that the objective that the app_name and app_uuid should be displayed when editing a product, but where will we fetch these from? And what is app_name here, is it similar to product name?
Thank you!

@stephanegigandet
Copy link
Contributor Author

@deveshidwivedi here is more details about the issue:

  • When you edit a product, at the bottom of the page, we display the list of changes to that product. e.g.
    image

  • This list is built from the changes.sto file that all products have. In it we may record a uuid sent by apps.

  • Currently when we have both a userid and a uuid, we display only the userid. This issue is to display both.

Related code is in lib/ProductOpener/Display.pm : sub display_product_history ($request_ref, $code, $product_ref) {

@anaritadauane
Copy link
Contributor

Hi @deveshidwivedi,

are you working on this issue?

@deveshidwivedi
Copy link
Contributor

Yes @anaritadauane thanks for asking

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Welcome to Open Food Facts. This issue should be approachable if you're new. Get in touch for help. product history We have kept 10 years of product revisions. This is useful to monitor edits & product improvements
Projects
None yet
5 participants