You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This PR implements user-consent-based error reporting functionality that allows users to opt-in to sending error reports to Sentry even when Sentry tracking is not globally enabled. When critical errors occur in authentication flows, users will be prompted to consent to sharing the error report to help improve the app.
Key improvements:
User consent mechanism for error reporting when Sentry is disabled
What is the reason for the change?
To provide users with the option to contribute to app improvement by sharing error reports when they encounter issues, while maintaining full control over their data sharing preferences.
What is the improvement/solution?
Implementation of a consent-based error reporting system that prompts users to voluntarily share error information when critical issues occur, enabling better debugging and app improvement without compromising user privacy.
Changelog
CHANGELOG entry: Added optional error reporting with user consent to help improve app stability
Related issues
Fixes:
Manual testing steps
<prerequisite: disable authentication API>
Ensure Sentry is disabled in app settings
Navigate to Login screen and trigger an error scenario
Verify that user is prompted to consent to sending error report
Test both "Send report" and "Try again" options in the consent dialog
Repeat for ChoosePassword and Onboarding error scenarios
Verify that error reports are only sent when user consents
Confirm that normal app functionality continues regardless of user choice
- Added OnboardingSheet component for user onboarding options.
- Integrated AccountStatus component to handle existing account scenarios.
- Updated navigation routes to include new onboarding flows.
- Enhanced ChoosePassword component to support OAuth login success handling.
- Updated localization files to include new strings for onboarding and account status.
- Added tests for new components and updated existing tests for integration.
This commit enhances the user experience during onboarding and account management by providing clear options and feedback for existing users.
Missing release label release-7.52.0 on PR. Adding release label release-7.52.0 on PR and removing other release labels(release-7.53.0), as PR was cherry-picked in branch 7.52.0.
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
area-onboardingINVALID-PR-TEMPLATEPR's body doesn't match templateNo QA NeededApply this label when your PR does not need any QA effort.no-changelogno-changelog Indicates no external facing user changes, therefore no changelog documentation neededrelease-7.52.0Issue or pull request that will be included in release 7.52.0team-onboardingOnboarding team
8 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
metamaskbot
added
the
release-7.53.0
Description
This PR implements user-consent-based error reporting functionality that allows users to opt-in to sending error reports to Sentry even when Sentry tracking is not globally enabled. When critical errors occur in authentication flows, users will be prompted to consent to sharing the error report to help improve the app.
Key improvements:
What is the reason for the change?
To provide users with the option to contribute to app improvement by sharing error reports when they encounter issues, while maintaining full control over their data sharing preferences.
What is the improvement/solution?
Implementation of a consent-based error reporting system that prompts users to voluntarily share error information when critical issues occur, enabling better debugging and app improvement without compromising user privacy.
Changelog
CHANGELOG entry: Added optional error reporting with user consent to help improve app stability
Related issues
Fixes:
Manual testing steps
<prerequisite: disable authentication API>
UI :