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

Integrate Nutri-Patrol at various relevant places in the app (tracker) #5390

Open
Tracked by #525
teolemon opened this issue Jun 16, 2024 · 5 comments
Open
Tracked by #525

Comments

@teolemon
Copy link
Member

What

  • Integrate Nutri-Patrol at various relevant places in the app (tracker)

Blocked on

Scopes

Where

  • Photo manager
  • Price Proof manager
  • Editing form
  • Product page
  • Search results

Why

  • Moderation of content

Part of

@teolemon
Copy link
Member Author

@g123k @monsieurtanuki NutriPatrol is now live in production

Tickets only available to moderators
Flagging available to everyone (logged-in)
Flagging webviews by Valentin:
A product https://nutripatrol.openfoodfacts.org/flag/product?barcode=3017620422003&source=web&flavor=off
A image https://nutripatrol.openfoodfacts.org/flag/image?barcode=3017620422003&source=web&flavor=off&image_id=1

We can do it in two times, integrate as links to webviews, then integrate natively at a later point. All the more that integrations need to be done at several places, and that's some work: #5390

@g123k
Copy link
Collaborator

g123k commented Jul 16, 2024

I have already discussed this with Valentin and it would be nice to let him implement it (and give him some help).

@Valimp
Copy link
Collaborator

Valimp commented Oct 29, 2024

Could you asign me on this issue please ?

@teolemon
Copy link
Member Author

teolemon commented Dec 1, 2024

@alexgarel
Copy link
Member

At the moment the NutriPatrol knowledge panel is not displayed for mobile. Ask for display as soon as the action is implemented in the mobile app. (it's to avoid to have an action that does nothing and can be frustrating for the users).

Maybe the mobile app itself should just not display actions knowledge panels it does not support (so that for users with older version they do not get confused) ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 💬 To discuss and validate
Development

No branches or pull requests

4 participants