Skip to content

Conversation

@lionellbriones
Copy link
Contributor

@lionellbriones lionellbriones commented Oct 28, 2025

Description

Fix error showing on UI on Shield plan page
image

Open in GitHub Codespaces

Changelog

CHANGELOG entry: N/A

Related issues

Fixes:

Manual testing steps

  1. Go to this page...

Screenshots/Recordings

Before

After

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

Note

Removes error handling and on-screen error messages from the Shield Plan page while keeping loading and pricing logic intact.

  • UI – ui/pages/shield-plan/shield-plan.tsx:
    • Remove error handling: stop destructuring error from useUserSubscriptions and useSubscriptionPricing, and delete the combined error variable.
    • Remove on-screen error message rendering; retain loading state and pricing/subscription logic.

Written by Cursor Bugbot for commit 2694f76. This will update automatically on new commits. Configure here.

@github-actions
Copy link
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@metamaskbot metamaskbot added the team-onboarding Onboarding team label Oct 28, 2025
@metamaskbot
Copy link
Collaborator

📊 Page Load Benchmark Results

Current Commit: fae877d | Date: 10/28/2025

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.06s (±42ms) 🟡 | historical mean value: 1.04s ⬆️ (historical data)
  • domContentLoaded-> current mean value: 734ms (±35ms) 🟢 | historical mean value: 728ms ⬆️ (historical data)
  • firstContentfulPaint-> current mean value: 79ms (±13ms) 🟢 | historical mean value: 80ms ⬇️ (historical data)
📈 Detailed Results
Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.06s 42ms 1.02s 1.32s 1.10s 1.32s
domContentLoaded 734ms 35ms 705ms 985ms 760ms 985ms
firstPaint 79ms 13ms 64ms 192ms 88ms 192ms
firstContentfulPaint 79ms 13ms 64ms 192ms 88ms 192ms
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms

Results generated automatically by MetaMask CI

@metamaskbot
Copy link
Collaborator

Builds ready [fae877d]
UI Startup Metrics (1259 ± 86 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyStandard HomeuiStartup1259112416048613061406
load107994514408611291218
domContentLoaded107494214358511211211
domInteractive201379111850
firstPaint56692127043110791175
backgroundConnect2322202597236251
firstReactRender26174663038
getState1883562230
initialActions41173512
loadScripts847722121185892985
setupStore1062431017
BrowserifyPower User HomeuiStartup24071908434377233964343
load1185950196633416191966
domContentLoaded1175942195532916141955
domInteractive301578225778
firstPaint795248196053010201960
backgroundConnect290226908162271908
firstReactRender25232922629
getState21815538146212381
initialActions183115356115
loadScripts921715148228213471482
setupStore1191831118
WebpackStandard HomeuiStartup8747431491948861061
load652595130790655817
domContentLoaded644589129989649810
domInteractive16125581542
firstPaint19759900197196665
backgroundConnect25124872838
firstReactRender281785103337
getState1262841420
initialActions3014248
loadScripts640587128587647799
setupStore1052331216
WebpackPower User HomeuiStartup18601551262736423452627
load73262410691458661069
domContentLoaded70460810211288231021
domInteractive24131232622123
firstPaint5017810273378361027
backgroundConnect11520386116270386
firstReactRender27244242742
getState19618623412200234
initialActions12147151947
loadScripts70060610111248111011
setupStore27896243696
FirefoxBrowserifyStandard HomeuiStartup14641280191511815121702
load1246110715578212921399
domContentLoaded1246110715578212921399
domInteractive1163127842125210
firstPaint------
backgroundConnect44241532649118
firstReactRender25215452638
getState73434712
initialActions3113235
loadScripts1218108914557612601359
setupStore136238231216
BrowserifyPower User HomeuiStartup28072322518770830705187
load14001162170914014821709
domContentLoaded14001162170914014811709
domInteractive17994429102266429
firstPaint------
backgroundConnect2452915343703931534
firstReactRender38305884558
getState1648224340197243
initialActions1611052615105
loadScripts13521138166513614461665
setupStore2451002823100
WebpackStandard HomeuiStartup15831372219314116531880
load13501193173510513921607
domContentLoaded13501193173510513921606
domInteractive1043535548109224
firstPaint------
backgroundConnect4322162194973
firstReactRender28225962938
getState104167171024
initialActions6121722415
loadScripts13241172170310213701569
setupStore177177211452
WebpackPower User HomeuiStartup29472474454062334614540
load15941330228627518752286
domContentLoaded15941330228527518742285
domInteractive1537537274187372
firstPaint------
backgroundConnect19934799248450799
firstReactRender50311011656101
getState18012026139202261
initialActions712161121
loadScripts15401264214624817992146
setupStore3761274161127
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 58 Bytes (0%)
  • ui: -117 Bytes (0%)
  • common: 10 Bytes (0%)

@lwin-kyaw lwin-kyaw added this pull request to the merge queue Oct 28, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 28, 2025
@lionellbriones lionellbriones added this pull request to the merge queue Oct 28, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 28, 2025
@lionellbriones lionellbriones added this pull request to the merge queue Oct 28, 2025
@lionellbriones lionellbriones removed this pull request from the merge queue due to a manual request Oct 28, 2025
@metamaskbot
Copy link
Collaborator

📊 Page Load Benchmark Results

Current Commit: 2694f76 | Date: 10/28/2025

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.03s (±44ms) 🟡 | historical mean value: 1.04s ⬇️ (historical data)
  • domContentLoaded-> current mean value: 720ms (±39ms) 🟢 | historical mean value: 728ms ⬇️ (historical data)
  • firstContentfulPaint-> current mean value: 76ms (±10ms) 🟢 | historical mean value: 79ms ⬇️ (historical data)
📈 Detailed Results
Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.03s 44ms 1.01s 1.35s 1.06s 1.35s
domContentLoaded 720ms 39ms 698ms 987ms 741ms 987ms
firstPaint 76ms 10ms 60ms 160ms 84ms 160ms
firstContentfulPaint 76ms 10ms 60ms 160ms 84ms 160ms
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms

Results generated automatically by MetaMask CI

@metamaskbot
Copy link
Collaborator

Builds ready [2694f76]
UI Startup Metrics (1249 ± 76 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyStandard HomeuiStartup1249110314467612941382
load107193912637411121199
domContentLoaded106593412537411071192
domInteractive20135791944
firstPaint62073118442610791152
backgroundConnect2302192607233242
firstReactRender27184363038
getState1979092130
initialActions50445516
loadScripts841718103373887968
setupStore962321013
BrowserifyPower User HomeuiStartup21191729385061627133850
load1070851187132814531871
domContentLoaded1059845185132014401851
domInteractive281467174767
firstPaint71314318845089151884
backgroundConnect27017410092063011009
firstReactRender272270112670
getState18513621619194216
initialActions53173517
loadScripts846676137425311771374
setupStore1382771427
WebpackStandard HomeuiStartup8567301081718671051
load63858688563649847
domContentLoaded63157987862642840
domInteractive161174101438
firstPaint18258881173190606
backgroundConnect24135482943
firstReactRender27175083236
getState1263841416
initialActions3014347
loadScripts62757786860640830
setupStore1051931215
WebpackPower User HomeuiStartup17421486240529517492405
load71960911181498291118
domContentLoaded686594943117774943
domInteractive191244103244
firstPaint47867947318778947
backgroundConnect94182949189294
firstReactRender25223022730
getState1809320425191204
initialActions15097252797
loadScripts681591933113764933
setupStore19659152759
FirefoxBrowserifyStandard HomeuiStartup15161313204513315761805
load1285113015779113451450
domContentLoaded1284113015779113451450
domInteractive1233642953137203
firstPaint------
backgroundConnect47252093150118
firstReactRender26215262741
getState9519919913
initialActions3113238
loadScripts1255110815518713161407
setupStore178218291322
BrowserifyPower User HomeuiStartup28462401371245134723712
load14231241167814616271678
domContentLoaded14231241167814616261678
domInteractive1736441582224415
firstPaint------
backgroundConnect20938717198428717
firstReactRender422981124781
getState17110422730193227
initialActions14177241077
loadScripts13701202159712514481597
setupStore25698253498
WebpackStandard HomeuiStartup16911493242512517321867
load1449126818489914871625
domContentLoaded1448126818489914861625
domInteractive1113039053120207
firstPaint------
backgroundConnect5125208246181
firstReactRender302383102942
getState9414114915
initialActions512022047
loadScripts1416125117519414541594
setupStore1374341419
WebpackPower User HomeuiStartup29602440394342233883943
load16051364185314917531853
domContentLoaded16051363185214917531852
domInteractive1357624352183243
firstPaint------
backgroundConnect23051648199382648
firstReactRender453170115270
getState18610431353228313
initialActions1311072512107
loadScripts15381326179614016891796
setupStore30574214374
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 58 Bytes (0%)
  • ui: -116 Bytes (0%)
  • common: 10 Bytes (0%)

@lionellbriones lionellbriones added this pull request to the merge queue Oct 28, 2025
Merged via the queue into main with commit e390537 Oct 28, 2025
174 checks passed
@lionellbriones lionellbriones deleted the feat/remove-errors-showing-on-ui branch October 28, 2025 17:49
@github-actions github-actions bot locked and limited conversation to collaborators Oct 28, 2025
@metamaskbot metamaskbot added the release-13.8.0 Issue or pull request that will be included in release 13.8.0 label Oct 28, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-13.8.0 Issue or pull request that will be included in release 13.8.0 size-S team-onboarding Onboarding team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants