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

Privacy Dashboard 8.0.0: Improved breakage form #3729

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

mgurgel
Copy link
Collaborator

@mgurgel mgurgel commented Jan 14, 2025

Task/Issue URL: https://app.asana.com/0/0/1209127604215386/f
CC: @jaceklyp

Description:

Upgrades Privacy Dashboard to 8.0.0, which contains the new site breakage flow as can be previewed here

Figma: https://www.figma.com/design/eeVdakW2pjijbDlv8HUSgZ/O-E---Report-Broken-Site-Flow-(Design-Updates-2024-09)?node-id=4024-6272&m=dev

Steps to test this PR:

  1. Trigger the breakage form in the two possible ways:
  • Open the Privacy Dashboard and click on "Report a problem with this site"
  • Open the app menu ••• and click on “Send Feedback > Report Broken Site” (macOS)
  1. Confirm that the breakage form matches the Figma or the web preview

  2. Attempt to complete a broken site report from both entry points above. Things to keep an eye on:

  • The “See what’s sent” toggle expands and collapses normally
  • When the category is “Something else”, trying to submit the form without a description should raise an alert
  • The description is optional for all other categories

Definition of Done:


Internal references:

Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation

Copy link
Contributor

@afterxleep afterxleep left a comment

Choose a reason for hiding this comment

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

Works as expected on macOS.

01-15 at 16 24

@mgurgel mgurgel force-pushed the mgurgel/improved-breakage-form branch from deff857 to 783a83f Compare January 17, 2025 14:25
@mgurgel mgurgel self-assigned this Jan 17, 2025
@mgurgel mgurgel force-pushed the mgurgel/improved-breakage-form branch from 783a83f to 7ea9a8f Compare January 17, 2025 14:54
Task/Issue URL:
https://app.asana.com/0/1206594217596623/1209173355503842/f

**Description**:

Set ups Privacy Dashboard EventMapping as to align with iOS. Each mapped
event fires a pixel when invoked by the Privacy Dashboard. See
duckduckgo/BrowserServicesKit#1167

Note: not all events are currently used by the Dashboard. Some are being
deprecated, which is why they might not be fired at all even though they
still exist in the mapping.

**Steps to test this PR**:
1. Invoke the breakage form in the two possible ways:
Open the Privacy Dashboard and click on "Report a problem with this
site"
Open the app menu ••• and click on "Report Broken Site” (iOS)

2. Confirm that the pixel `m_report-broken-site_shown` was fired
3. Submit a report
4. Confirm that the pixels `epbf_macos_desktop` and
`m_report-broken-site_sent` were fired
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants