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: show localized snap name in snap tag #29049

Merged
merged 10 commits into from
Jan 9, 2025

Conversation

montelaidev
Copy link
Contributor

@montelaidev montelaidev commented Dec 10, 2024

Description

This PR fixes the issue where the localized snap name is not being used when in the account tag. It also removes tech debt which is the mergeAccount function that isn't needed any more.

Open in GitHub Codespaces

Related issues

Fixes: https://github.com/MetaMask/accounts-planning/issues/769

Manual testing steps

  1. Install the BTC snap
  2. Create a BTC account
  3. Click on the account menu and see that the name is correct.

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.

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.

@montelaidev montelaidev force-pushed the fix/show-localized-snap-name-in-tag branch from 208b746 to 000eab4 Compare December 10, 2024 12:07
@montelaidev montelaidev added team-accounts team-ai AI team (for the Devin AI bot) and removed team-ai AI team (for the Devin AI bot) labels Dec 10, 2024
@montelaidev montelaidev marked this pull request as ready for review December 10, 2024 13:15
@montelaidev montelaidev requested a review from a team as a code owner December 10, 2024 13:15
@metamaskbot
Copy link
Collaborator

Builds ready [37ed0cb]
Page Load Metrics (1935 ± 66 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint17122273193713464
domContentLoaded16992208190213063
load17132280193513866
domInteractive249943189
backgroundConnect882332311
firstReactRender149522178
getState873851466933
initialActions01000
loadScripts13301749149211053
setupStore67612157
uiStartup192728422273232111
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: -1.76 KiB (-0.02%)
  • common: 0 Bytes (0.00%)

@metamaskbot
Copy link
Collaborator

Builds ready [4d6a2f3]
Page Load Metrics (1791 ± 77 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint23121921741385185
domContentLoaded14982077175014570
load15062135179116077
domInteractive2510341178
backgroundConnect6118483216
firstReactRender17179574220
getState5182354321
initialActions01000
loadScripts11281592131710751
setupStore687212412
uiStartup171327452139312150
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: -1.76 KiB (-0.02%)
  • common: 0 Bytes (0.00%)

@metamaskbot
Copy link
Collaborator

Builds ready [555f288]
Page Load Metrics (1843 ± 96 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint24323051756400192
domContentLoaded15732243181719393
load15942311184319996
domInteractive26214654522
backgroundConnect86828209
firstReactRender1694523014
getState681222412
initialActions01000
loadScripts11531755135016077
setupStore65917178
uiStartup174627902154274132

Copy link
Contributor

@vinnyhoward vinnyhoward left a comment

Choose a reason for hiding this comment

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

LGTM

@montelaidev montelaidev added this pull request to the merge queue Jan 8, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 8, 2025
@gantunesr gantunesr added this pull request to the merge queue Jan 9, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 9, 2025
@gantunesr gantunesr added this pull request to the merge queue Jan 9, 2025
Merged via the queue into main with commit a36a42c Jan 9, 2025
76 checks passed
@gantunesr gantunesr deleted the fix/show-localized-snap-name-in-tag branch January 9, 2025 14:15
@github-actions github-actions bot locked and limited conversation to collaborators Jan 9, 2025
@metamaskbot metamaskbot added the release-12.11.0 Issue or pull request that will be included in release 12.11.0 label Jan 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
release-12.11.0 Issue or pull request that will be included in release 12.11.0 team-accounts
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants