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

Remove native implementation of the site brekage reporting screen #5483

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

lmac012
Copy link
Contributor

@lmac012 lmac012 commented Jan 17, 2025

Task/Issue URL: https://app.asana.com/0/488551667048375/1208540813186997/f

Description

This PR removes native implementation of the site breakage reporting screen, which is currently hidden by a feature flag. We migrated to the web version of that form for several months and don't intend to go back to using native one.

Steps to test this PR

Navigate to the screen through Privacy Dashboard

  • Navigate to any website
  • Tap shield icon in the toolbar
  • Tap "Website not working?"
  • Verify that you landed on site breakage report screen and can send the report

Navigate to the screen through the options menu

  • Navigate to any website
  • Tap "Report Broken Site" in the options menu
  • Verify that you landed on site breakage report screen and can send the report

Navigate to the screen through the CTA shown on refreshing

  • Navigate to any website
  • Refresh 3 times in <20s
  • Tap "Report Broken Site" on the CTA that appears
  • Verify that you landed on site breakage report screen and can send the report

No UI changes

Copy link
Contributor Author

lmac012 commented Jan 17, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@lmac012 lmac012 marked this pull request as ready for review January 17, 2025 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant