Skip to content
This repository was archived by the owner on Oct 22, 2024. It is now read-only.

feat: implement metametrics opt-in page#436

Merged
OGPoyraz merged 1 commit intomainfrom
feat/401-ask-the-desktop-app-user-if-hes-fine-to-send-his-data-to-segment-sentry
Jan 20, 2023
Merged

feat: implement metametrics opt-in page#436
OGPoyraz merged 1 commit intomainfrom
feat/401-ask-the-desktop-app-user-if-hes-fine-to-send-his-data-to-segment-sentry

Conversation

@OGPoyraz
Copy link
Copy Markdown
Member

@OGPoyraz OGPoyraz commented Jan 18, 2023 •
edited
Loading

Overview

  • Implements MetaMetrics opt-in page and privacy tab
    Screenshot 2023-01-18 at 14 35 31
    Screenshot 2023-01-18 at 14 35 22

@OGPoyraz OGPoyraz requested a review from a team January 18, 2023 13:34
@OGPoyraz OGPoyraz linked an issue Jan 18, 2023 that may be closed by this pull request
Ask the desktop app user if he's fine to send his data to Segment & Sentry #401
Closed
@OGPoyraz OGPoyraz force-pushed the feat/401-ask-the-desktop-app-user-if-hes-fine-to-send-his-data-to-segment-sentry branch 3 times, most recently from 1f81c6b to f49dfd9 Compare January 19, 2023 12:27
@OGPoyraz OGPoyraz force-pushed the feat/401-ask-the-desktop-app-user-if-hes-fine-to-send-his-data-to-segment-sentry branch from f49dfd9 to a5d7ca5 Compare January 19, 2023 20:40
bergarces
bergarces approved these changes Jan 20, 2023
View reviewed changes
Copy link
Copy Markdown
Contributor

@bergarces bergarces left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@OGPoyraz OGPoyraz merged commit 32245f6 into main Jan 20, 2023
@OGPoyraz OGPoyraz deleted the feat/401-ask-the-desktop-app-user-if-hes-fine-to-send-his-data-to-segment-sentry branch February 1, 2023 07:48
@cryptotavares cryptotavares mentioned this pull request Mar 1, 2023
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@bergarces bergarces bergarces approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Ask the desktop app user if he's fine to send his data to Segment & Sentry

2 participants

@OGPoyraz @bergarces