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

fix: Remove obsolete PhishingController state #26308

Merged
merged 1 commit into from
Aug 2, 2024

Conversation

Gudahtt
Copy link
Member

@Gudahtt Gudahtt commented Aug 2, 2024

Description

Sentry reports showed cases of obsolete state in the PhishingController state. This obsolete state has been removed.

Open in GitHub Codespaces

Related issues

Fixes #26307

Manual testing steps

N/A

Screenshots/Recordings

N/A

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.

Copy link

codecov bot commented Aug 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.04%. Comparing base (69e57d8) to head (3db1130).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #26308      +/-   ##
===========================================
+ Coverage    70.04%   70.04%   +0.01%     
===========================================
  Files         1411     1411              
  Lines        50022    50031       +9     
  Branches     13809    13812       +3     
===========================================
+ Hits         35033    35042       +9     
  Misses       14989    14989              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Base automatically changed from remove-obsolete-network-controller-state to develop August 2, 2024 13:15
@metamaskbot
Copy link
Collaborator

Builds ready [b6903c1]
Page Load Metrics (251 ± 250 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint781871192713
domContentLoaded108427178
load462095251521250
domInteractive108427178
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 744 Bytes (0.02%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

Sentry reports showed cases of obsolete state in the PhishingController
state. This obsolete state has been removed.

Fixes #26307
@Gudahtt Gudahtt force-pushed the remove-obsolete-phishing-controller-state branch from b6903c1 to 3db1130 Compare August 2, 2024 13:35
@Gudahtt Gudahtt marked this pull request as ready for review August 2, 2024 13:36
@Gudahtt Gudahtt requested a review from a team as a code owner August 2, 2024 13:36
Copy link

sonarqubecloud bot commented Aug 2, 2024

@Gudahtt Gudahtt added team-extension-platform needs-dev-review PR needs reviews from other engineers (in order to receive required approvals) labels Aug 2, 2024
@metamaskbot
Copy link
Collaborator

Builds ready [3db1130]
Page Load Metrics (259 ± 267 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint683991348239
domContentLoaded1098292010
load481983259557267
domInteractive1098292010
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 348 Bytes (0.01%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

Copy link
Contributor

@desi desi left a comment

Choose a reason for hiding this comment

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

LGTM

@Gudahtt Gudahtt merged commit 5b56034 into develop Aug 2, 2024
79 checks passed
@Gudahtt Gudahtt deleted the remove-obsolete-phishing-controller-state branch August 2, 2024 14:46
@github-actions github-actions bot locked and limited conversation to collaborators Aug 2, 2024
@github-actions github-actions bot removed the needs-dev-review PR needs reviews from other engineers (in order to receive required approvals) label Aug 2, 2024
@metamaskbot metamaskbot added the release-12.4.0 Issue or pull request that will be included in release 12.4.0 label Aug 2, 2024
@gauthierpetetin gauthierpetetin added release-12.3.0 Issue or pull request that will be included in release 12.3.0 and removed release-12.4.0 Issue or pull request that will be included in release 12.4.0 labels Sep 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
release-12.3.0 Issue or pull request that will be included in release 12.3.0 team-extension-platform team-wallet-framework
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Error: No metadata found for 'listState'
6 participants