Skip to content

Conversation

@FrederikBolding
Copy link
Member

@FrederikBolding FrederikBolding commented Oct 9, 2023

Description

The SubjectMetadataController and its utilities were moved to the permission-controller package a while back (MetaMask/core#1234). This PR removes usage of the old deprecated package in favor of permission-controller. There should be no functionality changes by this change.

@FrederikBolding FrederikBolding added the team-snaps-deprecated DEPRECATED: please use "team-core-platform" instead label Oct 9, 2023
@FrederikBolding FrederikBolding requested review from a team as code owners October 9, 2023 09:36
@socket-security
Copy link

Updated and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Packages Version New capabilities Transitives Size Publisher
@metamask/controller-utils 3.4.0...5.0.1 None +0/-0 111 kB metamaskbot

🚮 Removed packages: @metamask/subject-metadata-controller@2.0.0

@metamaskbot
Copy link
Collaborator

Builds ready [5db93ee]
Page Load Metrics (734 ± 403 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint95155119199
domContentLoaded701641072612
load881939734839403
domInteractive701641072612
Bundle size diffs [🚀 Bundle size reduced!]
  • background: -420 Bytes (-0.01%)
  • ui: -90 Bytes (-0.00%)
  • common: -12.18 KiB (-0.25%)

@codecov
Copy link

codecov bot commented Oct 9, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (487c079) 68.64% compared to head (5db93ee) 68.65%.
Report is 2 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop   #21254   +/-   ##
========================================
  Coverage    68.64%   68.65%           
========================================
  Files         1017     1017           
  Lines        40792    40801    +9     
  Branches     10896    10893    -3     
========================================
+ Hits         28001    28008    +7     
- Misses       12791    12793    +2     
Files Coverage Δ
app/scripts/metamask-controller.js 54.04% <100.00%> (+0.11%) ⬆️
app/scripts/migrations/069.js 100.00% <ø> (ø)
...ect-header/permissions-connect-header.component.js 90.00% <ø> (ø)
ui/hooks/useOriginMetadata.js 78.57% <ø> (ø)
...rmissions-connect/permissions-connect.component.js 2.41% <ø> (ø)
...rmissions-connect/permissions-connect.container.js 6.45% <ø> (ø)
ui/selectors/selectors.js 86.84% <ø> (ø)

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

@FrederikBolding FrederikBolding merged commit 5236e02 into develop Oct 9, 2023
@FrederikBolding FrederikBolding deleted the fb/remove-subject-metadata-controller-pkg branch October 9, 2023 10:48
@github-actions github-actions bot locked and limited conversation to collaborators Oct 9, 2023
@metamaskbot metamaskbot added the release-11.4.0 Issue or pull request that will be included in release 11.4.0 label Oct 9, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-11.4.0 Issue or pull request that will be included in release 11.4.0 team-snaps-deprecated DEPRECATED: please use "team-core-platform" instead

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants