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

[FLASK] Bump snaps packages #20567

Merged
merged 10 commits into from
Sep 13, 2023
Merged

[FLASK] Bump snaps packages #20567

merged 10 commits into from
Sep 13, 2023

Conversation

FrederikBolding
Copy link
Member

@FrederikBolding FrederikBolding commented Aug 23, 2023

Explanation

Bumps Snaps Flask packages to the latest version and handles any breaking changes.

This PR also adds support for snap_getLocale.

@FrederikBolding FrederikBolding added flask team-snaps DEPRECATED: Use "team-snaps-platform" or "team-snaps-ecosystem" instead labels Aug 23, 2023
@metamaskbot
Copy link
Collaborator

Builds ready [947cb0e]
Page Load Metrics (1829 ± 93 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint1172001562311
domContentLoaded14792338182819091
load14792362182919493
domInteractive14792338182819091
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 361 Bytes (0.01%)
  • ui: 165 Bytes (0.00%)
  • common: 40.01 KiB (0.88%)

…20786)

* Move the `saveSnapKeyring` callback to the `SnapKeyring` constructor

This commit removes `saveSnapKeyring` from the list of callbacks passed
to the Snap Controller since this function contains business logic
specific to the Snap Keyring.

Instead, `saveSnapKeyring` is injected into the Snap Keyring through a
new argument called `callbacks`.

In the future, the `callbacks` argument can be reused to pass other
callbacks.

* Update the Snap Keyring dependency

* chore: use release versions of `eth-snap-keyring` and `rpc-methods`

* chore: update `yarn.lock`

* chore: update `yarn.lock`
@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.

@codecov
Copy link

codecov bot commented Sep 12, 2023

Codecov Report

Patch coverage: 12.50% and no project coverage change.

Comparison is base (37824db) 68.20% compared to head (56310e5) 68.20%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #20567      +/-   ##
===========================================
- Coverage    68.20%   68.20%   -0.00%     
===========================================
  Files          999      999              
  Lines        39846    39855       +9     
  Branches     10669    10671       +2     
===========================================
+ Hits         27174    27180       +6     
- Misses       12672    12675       +3     
Files Changed Coverage Δ
shared/constants/permissions.ts 100.00% <ø> (ø)
shared/constants/snaps/permissions.ts 100.00% <ø> (ø)
ui/helpers/utils/permission.js 16.13% <0.00%> (-0.18%) ⬇️
app/scripts/metamask-controller.js 54.44% <14.29%> (+0.08%) ⬆️

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

@FrederikBolding FrederikBolding marked this pull request as ready for review September 12, 2023 10:16
@FrederikBolding FrederikBolding requested review from a team as code owners September 12, 2023 10:16
@metamaskbot
Copy link
Collaborator

Builds ready [56310e5]
Page Load Metrics (2196 ± 170 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint1183642085627
domContentLoaded148628502196353170
load148628502196353170
domInteractive148628502196353170
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 361 Bytes (0.01%)
  • ui: 165 Bytes (0.00%)
  • common: 40.01 KiB (0.88%)

@FrederikBolding FrederikBolding merged commit f19e1f5 into develop Sep 13, 2023
@FrederikBolding FrederikBolding deleted the fb/snaps-flask-bump branch September 13, 2023 15:14
@github-actions github-actions bot locked and limited conversation to collaborators Sep 13, 2023
@metamaskbot metamaskbot added the release-11.2.0 Issue or pull request that will be included in release 11.2.0 label Sep 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
flask release-11.2.0 Issue or pull request that will be included in release 11.2.0 team-snaps DEPRECATED: Use "team-snaps-platform" or "team-snaps-ecosystem" instead
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants