Skip to content

refactor: multichain activity tab #31439

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

Merged
merged 17 commits into from
Apr 1, 2025
Merged

refactor: multichain activity tab #31439

merged 17 commits into from
Apr 1, 2025

Conversation

darioAnongba
Copy link
Contributor

@darioAnongba darioAnongba commented Mar 31, 2025

Description

Align activity tab chain agnostic algorithm, aligned with expected transaction metadata from listAccountTransactions.

  • Use the to array for Send transactions amounts
  • Use a different decimal threshold per network
  • Aggregate amounts per token
  • Aggregate amounts to compute the Amount
  • Show a list of addresses in From field for Receive transactions
  • Remove search in transaction to compute amounts in Swaps.
  • Support for Redeposit transactions
  • Fixes Storybook

Open in GitHub Codespaces

Related issues

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

Manual testing steps

  1. Create Solana or Bitcoin account
  2. Send and receive transactions (can also send to yourself).
  3. Display activity tab

Screenshots/Recordings

Before

Screenshot 2025-03-31 at 18 22 07

After

Screenshot 2025-03-31 at 18 14 32

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.

@darioAnongba darioAnongba self-assigned this Mar 31, 2025
@darioAnongba darioAnongba marked this pull request as draft March 31, 2025 16:17
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
Copy link
Collaborator

Builds ready [2349821]
UI Startup Metrics (1243 ± 67 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1243111415156712841347
load10869701288611150986
domContentLoaded10809651280611155989
domInteractive16133141627
firstPaint75878128643024278
backgroundConnect106253910
firstReactRender21154672039
getState11537769
initialActions001001
loadScripts857751103860901942
setupStore8431379
WebpackHomeuiStartup977722145997975992
load833544112676868925
domContentLoaded826540111375861921
domInteractive161292111441
firstPaint53071952345857895
backgroundConnect16115281539
firstReactRender15123941428
getState7414279
initialActions001001
loadScripts824530110174860912
setupStore7513289
FirefoxBrowserifyHomeuiStartup13541143191516413641794
load12221026177815912441619
domContentLoaded12221026177715912441619
domInteractive9736204289098
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect21164042331
firstReactRender22192832427
getState7436478
initialActions001001
loadScripts12011010175615712211596
setupStore6328467
WebpackHomeuiStartup10018541553159901973
load8777441348142810921
domContentLoaded8767441347142810921
domInteractive115362002714595
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect211376112439
firstReactRender19162621924
getState94621089
initialActions001001
loadScripts8597251318138797954
setupStore9565978
Bundle size diffs [🚀 Bundle size reduced!]
  • background: -9.84 KiB (-0.17%)
  • ui: 167 Bytes (0%)
  • common: 9.54 KiB (0.1%)

@metamaskbot
Copy link
Collaborator

Builds ready [dbacb12]
UI Startup Metrics (1214 ± 69 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1214107713716912661330
load10639411205671205991
domContentLoaded1057937119967937987
domInteractive16134151630
firstPaint816931205387269984
backgroundConnect97282910
firstReactRender19134451930
getState11443868
initialActions001001
loadScripts84272797966890952
setupStore8517278
WebpackHomeuiStartup1020761128891966996
load857609109687918963
domContentLoaded852605109287914960
domInteractive16134671435
firstPaint3445997332883496
backgroundConnect18135281641
firstReactRender17123041727
getState5317358
initialActions001000
loadScripts849596109087911958
setupStore8516379
FirefoxBrowserifyHomeuiStartup13861183192915114101771
load12481063177614412831594
domContentLoaded12481063177614412821594
domInteractive10838207328598
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2416127132440
firstReactRender23194142530
getState7423378
initialActions001001
loadScripts12251045174214212581570
setupStore5412267
WebpackHomeuiStartup9618161405163870982
load8427121216147774887
domContentLoaded8417121215147773887
domInteractive118671962313495
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect19127482028
firstReactRender18162821823
getState84751078
initialActions001001
loadScripts8256991189142760872
setupStore114721478
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 0 Bytes (0%)
  • ui: 58 Bytes (0%)
  • common: 313 Bytes (0%)

@darioAnongba darioAnongba marked this pull request as ready for review April 1, 2025 11:42
@metamaskbot
Copy link
Collaborator

Builds ready [4d50f36]
UI Startup Metrics (1224 ± 55 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1224110314315512591311
load10659381172501135987
domContentLoaded10579311166501137986
domInteractive16133241628
firstPaint8061561174397240986
backgroundConnect116709910
firstReactRender20154552034
getState11430768
initialActions001000
loadScripts83971395451876922
setupStore8519379
WebpackHomeuiStartup10326931562102971995
load870521109871898959
domContentLoaded864512109271896970
domInteractive16134771436
firstPaint32656100133684091
backgroundConnect18136291742
firstReactRender16123251728
getState6319456
initialActions001001
loadScripts861511107570894960
setupStore8525379
FirefoxBrowserifyHomeuiStartup13831185182413914371721
load12461061167513213021540
domContentLoaded12451061167513213021540
domInteractive9942216278997
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect22164352433
firstReactRender23196252528
getState6412179
initialActions0028301
loadScripts12251045165513012851513
setupStore6318268
WebpackHomeuiStartup9608131551167883979
load8427061356148786901
domContentLoaded8417061356148785901
domInteractive124771902214177
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect201284102034
firstReactRender18163021824
getState1047213709
initialActions001001
loadScripts8256971335144772887
setupStore8592978
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 6 Bytes (0%)
  • ui: 666 Bytes (0.01%)
  • common: 313 Bytes (0%)

zone-live
zone-live previously approved these changes Apr 1, 2025
Copy link
Contributor

@zone-live zone-live left a comment

Choose a reason for hiding this comment

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

LGTM 💪🏼

@metamaskbot
Copy link
Collaborator

Builds ready [9325a79]
UI Startup Metrics (1196 ± 72 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1196107214587212351319
load1041933125863957992
domContentLoaded1035928125063953992
domInteractive16132941627
firstPaint7751461239389239985
backgroundConnect116768910
firstReactRender19153942027
getState11534768
initialActions001001
loadScripts820718102462847922
setupStore8520278
WebpackHomeuiStartup1020834128184959992
load85761299167904948
domContentLoaded85260598366894944
domInteractive16124161433
firstPaint2995794329680927
backgroundConnect18134271640
firstReactRender16123141727
getState6220358
initialActions001001
loadScripts84960297366890942
setupStore8417379
FirefoxBrowserifyHomeuiStartup14131196188014814411779
load12661075172213312931587
domContentLoaded12661075172113312931587
domInteractive10341198288797
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect24177482532
firstReactRender23193532328
getState7437479
initialActions001001
loadScripts12431058169813212741562
setupStore6412268
WebpackHomeuiStartup10018321511152904974
load8737271291134819970
domContentLoaded8737271289134820973
domInteractive116331922514697
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2213116142541
firstReactRender19163431925
getState104761189
initialActions001001
loadScripts8557151272130813979
setupStore9451889
Bundle size diffs [🚀 Bundle size reduced!]
  • background: 0 Bytes (0%)
  • ui: 61 Bytes (0%)
  • common: -3.94 KiB (-0.04%)

@darioAnongba darioAnongba added this pull request to the merge queue Apr 1, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 1, 2025
@darioAnongba darioAnongba added this pull request to the merge queue Apr 1, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 1, 2025
@darioAnongba darioAnongba added this pull request to the merge queue Apr 1, 2025
Merged via the queue into main with commit ebd5018 Apr 1, 2025
147 checks passed
@darioAnongba darioAnongba deleted the refactor/activity-tab branch April 1, 2025 20:28
@github-actions github-actions bot locked and limited conversation to collaborators Apr 1, 2025
@metamaskbot metamaskbot added the release-12.17.0 Issue or pull request that will be included in release 12.17.0 label Apr 1, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
release-12.17.0 Issue or pull request that will be included in release 12.17.0 team-accounts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants