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.

Sign up for GitHub

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

Jump to bottom

Log whenever people encounter an unknown product using Matomo #2852

Closed
teolemon opened this issue Aug 28, 2022 · 6 comments · Fixed by #2854
Closed

Log whenever people encounter an unknown product using Matomo #2852

teolemon opened this issue Aug 28, 2022 · 6 comments · Fixed by #2854
Assignees
@monsieurtanuki
Labels
📈 Analytics We use Sentry and Matomo, with an opt-in system Product addition The easier it is to add a product and get Nutri-Score, Eco-Score, the happier the users. Sentry

Comments

@teolemon
Copy link
Member

teolemon commented Aug 28, 2022 •
edited

What

  • Log whenever people encounter an unknown product using Matomo

Why

  • Comments like these: Disaster, out of five products, two recognized and lack of information. The competition does better. The old version worked better.
  • I suspect it's a scan issue, but not 100% sure. It would allow to rule out lack of products
  • It would also allow to get a ratio of people who actually decide to add the product

Unrelated

  • by the way @g123k, the complain is build 722 (after fixes) on a Pixel 5

Part of
@teolemon teolemon added Product addition The easier it is to add a product and get Nutri-Score, Eco-Score, the happier the users. 📈 Analytics We use Sentry and Matomo, with an opt-in system Sentry labels Aug 28, 2022
@monsieurtanuki
Copy link
Contributor

@teolemon Regarding "unknown products", it may come from the country and language filters.

@teolemon
Copy link
Member Author

I mean in the scan screen. Encounter the Unknown product card, in the context of the scan screen.

@monsieurtanuki
Copy link
Contributor

I guess we filter country and language in the scan screen too. Shouldn't we?
Or the point here is just to send a message to Sentry when we scan an unknown product, and that's it.

@teolemon
Copy link
Member Author

just to send a message to Sentry when we scan an unknown product

@monsieurtanuki monsieurtanuki self-assigned this Aug 28, 2022
monsieurtanuki added a commit to monsieurtanuki/smooth-app that referenced this issue Aug 28, 2022
@monsieurtanuki
feat: openfoodfacts#2852 - Sentry message when barcode is not found
8c5e06d 
Impacted file:
* `barcode_product_query.dart`: added Sentry message when barcode is not found
@monsieurtanuki monsieurtanuki mentioned this issue Aug 28, 2022
Merged
monsieurtanuki added a commit to monsieurtanuki/smooth-app that referenced this issue Aug 28, 2022
@monsieurtanuki
Merge branch 'develop' into feat/openfoodfacts#2852
5fd124a
@teolemon
Copy link
Member Author

Oh shit, I meant Matomo :-(

@teolemon teolemon changed the title Log whenever people encounter an unknown product using Sentry Log whenever people encounter an unknown product using Matomo Aug 28, 2022
monsieurtanuki added a commit to monsieurtanuki/smooth-app that referenced this issue Aug 28, 2022
@monsieurtanuki
feat: openfoodfacts#2852 - Matomo message when barcode is not found
44446c1 
Impacted files:
* `analytics_helper.dart`: new method about barcode not found; fixed typos
* `barcode_product_query.dart`: added Matomo (and not Sentry) message when barcode is not found
monsieurtanuki added a commit to monsieurtanuki/smooth-app that referenced this issue Aug 28, 2022
@monsieurtanuki
Merge remote-tracking branch 'origin/feat/openfoodfacts#2852' into feat/
c0989fe 
openfoodfacts#2852
@monsieurtanuki
Copy link
Contributor

Oh shit, I meant Matomo :-(

https://www.youtube.com/watch?v=dPOyOM7wxlE&t=140s

monsieurtanuki added a commit that referenced this issue Aug 29, 2022
@monsieurtanuki
feat: #2852 - Matomo message when barcode is not found (#2854)
3dca648 
Impacted files:
* `analytics_helper.dart`: new method about barcode not found; fixed typos
* `barcode_product_query.dart`: added Matomo (and not Sentry) message when barcode is not found
@github-actions github-actions bot mentioned this issue Aug 29, 2022
Merged
@github-actions github-actions bot mentioned this issue Sep 6, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@monsieurtanuki monsieurtanuki

Labels
📈 Analytics We use Sentry and Matomo, with an opt-in system Product addition The easier it is to add a product and get Nutri-Score, Eco-Score, the happier the users. Sentry
Projects
🤳🥫 The Open Food Facts mobile app (Android & iOS)
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: #2852 - Matomo message when barcode is not found monsieurtanuki/smooth-app
2 participants
@teolemon @monsieurtanuki