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

Add metrics tracking notice to IPFS Companion #1114

Closed
Tracked by #1115 ...
juliaxbow opened this issue Dec 14, 2022 · 4 comments
Closed
Tracked by #1115 ...

Add metrics tracking notice to IPFS Companion #1114

juliaxbow opened this issue Dec 14, 2022 · 4 comments
Assignees
Labels
kind/discussion Topical discussion; usually not changes to codebase kind/maintenance Work required to avoid breaking changes or harm to project's status quo kind/tracking A meta-issue for tracking work P0 Critical: Tackled by core team ASAP status/ready Ready to be worked topic/design-ux UX strategy, research, not solely visual design

Comments

@juliaxbow
Copy link

Background:
Following efforts to track and obtain necessary metrics around IPFS usage, we must notify companion users about how data is being collected and used as well as easily allow for users to opt out of non-necessary metrics

Proposed solution

  • One way to do this is to have a push notification in the companion extension. Chrome and Firefox have requirements for how these are formatted; see screen recording for proposal
  • Pros: One-time alert that defaults to tracking user metrics; user would click on the alert or gear icon to go to preferences to update settings in preferences window. A push notification makes the change in how we collect metrics super visible.
  • Cons: Users may have push notifications disabled through their browser. Push notifications are also generally found to be annoying
Push.Notification.companion.mov

Alternative Solutions

In all options, the actual toggles for cookie opt-ins/out live in preferences. The alternatives below describe methods of notifying the user.

Description: Notification alert on gear icon for initial notice of cookie tracking. Clicking on this will bring the user to the user preferences, where they will see a section that allows them to update their cookie preferences. The notification will disappear after first click.
Gear Notification

Description: Cookie link in extension window. Always include link at bottom of the extension window that will link to user preferences and allow the user to change cookie preferences.
IPFS Cookie

Additional context

@juliaxbow juliaxbow added the need/triage Needs initial labeling and prioritization label Dec 14, 2022
@welcome

This comment was marked as resolved.

@lidel
Copy link
Member

lidel commented Dec 18, 2022

Somethign to be cognizant, is notification/monit overload.
EU people are way pass the cookie-popup fatigue.
Whatever we do should minimize the annoyance, and not cruft the user interface.

  • one-time "Notification alert on gear icon" sounds good 👍
  • @juliaxbow food for thought: alternative idea is to add notice about "Opt-out telemetry" to the welcome screen that is presented after initial install. Either at the bottom, or as an additional section on the right (above or below videos):

    2022-12-18_23-53

If you want to re-open the welcome screen, click on the cube in the left top corner of the Companion's browser action menu.

@whizzzkid whizzzkid added kind/discussion Topical discussion; usually not changes to codebase kind/maintenance Work required to avoid breaking changes or harm to project's status quo topic/design-ux UX strategy, research, not solely visual design need/analysis Needs further analysis before proceeding need/author-input Needs input from the original author need/community-input Needs input from the wider community kind/tracking A meta-issue for tracking work and removed need/triage Needs initial labeling and prioritization labels Dec 19, 2022
@whizzzkid
Copy link
Contributor

@SgtPooki since you have a PR out there, I think you can take on this issue!

@SgtPooki SgtPooki added status/ready Ready to be worked P0 Critical: Tackled by core team ASAP and removed need/analysis Needs further analysis before proceeding need/author-input Needs input from the original author need/community-input Needs input from the wider community labels Jan 3, 2023
@SgtPooki
Copy link
Member

discussion with @lidel. We're going to close this as this is not the plan we're moving forward with. see https://pl-strflt.notion.site/Telemetry-2023Q1-b005d4f217f44db3986902c67d922cf4 for latest

@github-project-automation github-project-automation bot moved this from Needs Grooming to Done in IPFS-GUI (PL EngRes) Jan 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/discussion Topical discussion; usually not changes to codebase kind/maintenance Work required to avoid breaking changes or harm to project's status quo kind/tracking A meta-issue for tracking work P0 Critical: Tackled by core team ASAP status/ready Ready to be worked topic/design-ux UX strategy, research, not solely visual design
Projects
No open projects
Archived in project
Development

No branches or pull requests

4 participants