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

Implement an analytics consent UI as a checkbox in the onboarding (or just after ?) #1222

Closed
1 task
Tracked by #561 ...
teolemon opened this issue Mar 16, 2022 · 10 comments · Fixed by #1310
Closed
1 task
Tracked by #561 ...

Implement an analytics consent UI as a checkbox in the onboarding (or just after ?) #1222

teolemon opened this issue Mar 16, 2022 · 10 comments · Fixed by #1310
Assignees
Labels
📈 Analytics We use Sentry and Matomo, with an opt-in system 🤗 Onboarding We need to onboard users on how the app works, but also on content like Nutri-Score, Eco-Score…

Comments

@teolemon
Copy link
Member

teolemon commented Mar 16, 2022

What

  • Implement an analytics consent UI as a checkbox in the onboarding (or just after ?)

V1 inspiration

image

image

image

@teolemon teolemon added 🤗 Onboarding We need to onboard users on how the app works, but also on content like Nutri-Score, Eco-Score… 📈 Analytics We use Sentry and Matomo, with an opt-in system labels Mar 16, 2022
@PrabeshPP
Copy link
Contributor

PrabeshPP commented Mar 21, 2022

hi @teolemon .Is anyone working on this? I would like to contribute. Also, in my opinion asking the permission while on onboarding screen would look nice.
Waiting for the reply!

@teolemon
Copy link
Member Author

Just assigned you @PrabeshPP
It's a legal requirement, but it makes the onboarding longer.
We'll probably have to rethink onboarding more deeply.

@PrabeshPP
Copy link
Contributor

hi @teolemon do I also need to write the logic for the Consent or just creating the UI for consent would be suffice for now.

@M123-dev
Copy link
Member

The logic is already implemented @PrabeshPP

Future<void> setCrashReports(final bool state) async =>

And the lines below

setCrashReport and setAnalyticsReport

@PrabeshPP
Copy link
Contributor

@teolemon @M123-dev I have done some prototype designing for the consent UI.

Have a look and give me some feedback as soon as possible.

@M123-dev
Copy link
Member

I like it @PrabeshPP, but then this is the only page in the onboarding which has the buttons on a different position which isn't that nice. But putting the accept button where normally the next button is also too intrusive.

Any opinions @teolemon @monsieurtanuki @cli1005

@teolemon
Copy link
Member Author

It looks reasonable, and having it in the same place would be a little too much dark pattern.

@teolemon
Copy link
Member Author

We can iterate the UI if needed / feedback from users

@monsieurtanuki
Copy link
Contributor

I would add

  • "meaningful" colors (like green and red) (I suppose your prototype tool is in grey tones in order to avoid color choice endless talks)
  • meaningful icons (like "v" and "x")

@teolemon
Copy link
Member Author

  • It's actually grey in v1 🙃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📈 Analytics We use Sentry and Matomo, with an opt-in system 🤗 Onboarding We need to onboard users on how the app works, but also on content like Nutri-Score, Eco-Score…
Development

Successfully merging a pull request may close this issue.

4 participants