-
Notifications
You must be signed in to change notification settings - Fork 5.4k
feat: added pna-25 banner #38112
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
feat: added pna-25 banner #38112
Conversation
Builds ready [3feccf9]
UI Startup Metrics (1168 ± 107 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
✨ Files requiring CODEOWNER review ✨🕵️ @MetaMask/extension-privacy-reviewers (1 files, +1 -0)
🧪 @MetaMask/qa (1 files, +1 -0)
🔐 @MetaMask/web3auth (2 files, +23 -2)
|
Builds ready [d270e72]
UI Startup Metrics (1340 ± 126 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
|
Should this PR include the metric for toast being displayed or is that a follow-up? From ticket: "Log an event for the notice having been displayed" |
Builds ready [3ff5436]
UI Startup Metrics (1233 ± 101 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Builds ready [8e6603e]
UI Startup Metrics (1202 ± 89 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Builds ready [ca876f7]
UI Startup Metrics (1271 ± 122 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Builds ready [7a157ea]
UI Startup Metrics (1231 ± 114 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Builds ready [bf7d334]
UI Startup Metrics (1231 ± 108 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
seaona
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
QA changes LGTM

This PR is to add the updated privacy notice banner on homepage
Feature Flow Diagram
Changelog
CHANGELOG entry: Introduces metametrics banner
Related issues
Fixes: issue
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Adds a feature-flagged MetaMetrics privacy banner (
PNA25) with persistedpna25Acknowledgedstate, onboarding/use-metrics integration, and supporting env/config/i18n updates.Pna25Bannerinui/components/app/toast-master/toast-master.jswith selectorselectShowPna25Bannerand action to acknowledge (setPna25Acknowledged).selectShowPna25Bannerintoast-master/selectors.ts; wiresetPna25Acknowledgedintoast-master/utils.ts.useEnableMetametricsauto-acknowledges whenextensionUxPna25enabled andpna25Acknowledged === false.metametrics.jsandwelcome.jssetpna25Acknowledgedfor new/social users whenEXTENSION_UX_PNA25is enabled; updated checkbox copy toggle.pna25AcknowledgedtoAppStateControllerstate, defaults, metadata, settersetPna25Acknowledged, and background API exposure viametamask-controller.js.app/scripts/constants/sentry-state.ts.shared/types/background.ts; selectorgetPna25Acknowledged.EXTENSION_UX_PNA25: trueacrossbuilds.ymland default envs.pna25BannerTitle,onboardingMetametricCheckboxDescriptionOneUpdatedinapp/_locales/en*/messages.json.METAMETRICS_SETTINGS_LINKinshared/lib/ui-utils.js.pna25Acknowledgedand related ordering tweaks.setPna25Acknowledgedinui/store/actions.tsand background wiring.Written by Cursor Bugbot for commit bf7d334. This will update automatically on new commits. Configure here.