feat: SOL-80 transaction details #29323

zone-live · 2024-12-18T14:46:54Z

Description

This PR adds the new transaction details modal for the non-evm networks, BTC and SOL.

Screen.Recording.2024-12-17.at.12.27.40.mov

Related issues

Fixes: https://consensyssoftware.atlassian.net/browse/SOL-80

Manual testing steps

Testing this is a bit extensive, but if you still want to give it a go these are the steps:

Checkout this branch and run yarn
Update the file shared/lib/accounts/solana-wallet-snap.ts with:
export const SOLANA_WALLET_SNAP_ID: SnapId = 'local:http://localhost:8080' as SnapId;
Update the filtering code in MultichainTransactionsController under node modules to return transactions for devnet, currently only returns for mainnet. It's under node_modules/@metamask/multichain-transactions-controller/dist/MultichainTransactionsController.mjs and node_modules/@metamask/multichain-transactions-controller/dist/MultichainTransactionsController.cjs with:

MultichainNetwork.SolanaDevnet
instead of
MultichainNetwork.Solana

Run the extension with yarn start:flask
Run the Snap: https://github.com/MetaMask/snap-solana-wallet
- Clone it
- Run yarn
- Run yarn start
Go to http://localhost:3000/
Install the Snap
In the extension, go to the Settings > Experimental > Enable Solana account
Create a Solana account from the account-list menu
Fund the new Solana account with some SOL, use a faucet like https://faucet.solana.com/
the initial Tx with funds from the faucet will display in the activity tab
Click in it and you will see the Tx details modal
Thats it! 🎉

Screenshots/Recordings

Before

Didn't exist.

After

Pre-merge author checklist

I've followed MetaMask Contributor Docs and MetaMask Extension Coding Standards.
I've completed the PR template to the best of my ability
I’ve included tests if applicable
I’ve documented my code using JSDoc format if applicable
I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

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.

…roller

…github.com:MetaMask/metamask-extension into SOL-46-adds-the-multichain-transactions-controller

…roller

github-actions · 2024-12-18T14:47:08Z

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.

…o SOL-80-transaction-details

...nents/app/multichain-transaction-details-modal/multichain-transaction-details-modal.test.tsx

ui/components/app/multichain-transaction-details-modal/multichain-transaction-details-modal.tsx

...nents/app/multichain-transaction-details-modal/multichain-transaction-details-modal.test.tsx

ui/components/app/multichain-transaction-details-modal/multichain-transaction-details-modal.tsx

...nents/app/multichain-transaction-details-modal/multichain-transaction-details-modal.test.tsx

…ain-transaction-details-modal.test.tsx Co-authored-by: Charly Chevalier <[email protected]>

metamaskbot · 2025-01-28T16:45:29Z

Builds ready [d6e121f]

builds: chrome, firefox
builds (flask): chrome, firefox
builds (MMI): chrome
builds (test): chrome, firefox
builds (test-flask): chrome, firefox
build viz: Build System
mv3: Bundle Size Stats
mv2: E2e Actions Stats
storybook: Storybook
typescript migration: Dashboard
all artifacts
bundle viz:
- background: 0, 1, 2, 3, 4, 5, 6, 7
- common: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13
- ui: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13
- content-script: 0
- offscreen: 0

Page Load Metrics (1850 ± 89 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	379	2197	1794	363	174
		domContentLoaded	1468	2183	1820	176	85
		load	1474	2204	1850	185	89
		domInteractive	26	83	38	15	7
		backgroundConnect	9	66	31	17	8
		firstReactRender	16	68	30	18	9
		getState	4	62	24	22	11
		initialActions	0	0	0	0	0
		loadScripts	1063	1655	1351	145	70
		setupStore	8	63	17	16	8
		uiStartup	1702	2461	2115	195	93

Bundle size diffs

background: 0 Bytes (0.00%)
ui: 0 Bytes (0.00%)
common: 0 Bytes (0.00%)

metamaskbot · 2025-01-29T13:54:45Z

Builds ready [c0116e2]

builds: chrome, firefox
builds (flask): chrome, firefox
builds (MMI): chrome
builds (test): chrome, firefox
builds (test-flask): chrome, firefox
build viz: Build System
mv3: Bundle Size Stats
mv2: E2e Actions Stats
storybook: Storybook
typescript migration: Dashboard
all artifacts
bundle viz:
- background: 0, 1, 2, 3, 4, 5, 6, 7
- common: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13
- ui: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13
- content-script: 0
- offscreen: 0

Page Load Metrics (1760 ± 110 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	1468	2285	1760	233	112
		domContentLoaded	1454	2277	1736	231	111
		load	1471	2285	1760	229	110
		domInteractive	19	73	32	11	5
		backgroundConnect	9	69	24	17	8
		firstReactRender	16	104	44	28	13
		getState	5	66	17	20	9
		initialActions	0	0	0	0	0
		loadScripts	1036	1621	1249	180	87
		setupStore	8	63	20	18	9
		uiStartup	1648	2617	2079	262	126

Bundle size diffs

background: 0 Bytes (0.00%)
ui: 0 Bytes (0.00%)
common: 0 Bytes (0.00%)

metamaskbot · 2025-01-29T15:09:27Z

Builds ready [d6d7a16]

builds: chrome, firefox
builds (flask): chrome, firefox
builds (MMI): chrome
builds (test): chrome, firefox
builds (test-flask): chrome, firefox
build viz: Build System
mv3: Bundle Size Stats
mv2: E2e Actions Stats
storybook: Storybook
typescript migration: Dashboard
all artifacts
bundle viz:
- background: 0, 1, 2, 3, 4, 5, 6, 7
- common: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13
- ui: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13
- content-script: 0
- offscreen: 0

Page Load Metrics (1613 ± 64 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	1429	1925	1614	134	64
		domContentLoaded	1421	1869	1588	126	61
		load	1429	1918	1613	134	64
		domInteractive	22	99	47	23	11
		backgroundConnect	8	83	29	22	10
		firstReactRender	15	74	36	24	11
		getState	5	50	13	13	6
		initialActions	0	0	0	0	0
		loadScripts	1005	1404	1145	114	55
		setupStore	8	58	16	17	8
		uiStartup	1602	2107	1833	139	67

Bundle size diffs

background: 0 Bytes (0.00%)
ui: 0 Bytes (0.00%)
common: 0 Bytes (0.00%)

zone-live · 2025-01-29T17:37:32Z

metamaskbot · 2025-01-29T18:07:16Z

Builds ready [3403374]

builds: chrome, firefox
builds (flask): chrome, firefox
builds (MMI): chrome
builds (test): chrome, firefox
builds (test-flask): chrome, firefox
build viz: Build System
mv3: Bundle Size Stats
mv2: E2e Actions Stats
storybook: Storybook
typescript migration: Dashboard
all artifacts
bundle viz:
- background: 0, 1, 2, 3, 4, 5, 6, 7
- common: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13
- ui: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13
- content-script: 0
- offscreen: 0

Page Load Metrics (1800 ± 93 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	282	2298	1734	389	187
		domContentLoaded	1535	2285	1764	195	94
		load	1544	2291	1800	194	93
		domInteractive	26	92	44	22	11
		backgroundConnect	10	90	39	29	14
		firstReactRender	16	99	43	27	13
		getState	5	64	16	15	7
		initialActions	0	0	0	0	0
		loadScripts	1113	1723	1304	175	84
		setupStore	9	91	33	26	13
		uiStartup	1770	2637	2107	277	133

Bundle size diffs

background: 0 Bytes (0.00%)
ui: 0 Bytes (0.00%)
common: 0 Bytes (0.00%)

metamaskbot · 2025-01-30T10:15:34Z

Builds ready [6996ee1]

builds: chrome, firefox
builds (flask): chrome, firefox
builds (MMI): chrome
builds (test): chrome, firefox
builds (test-flask): chrome, firefox
build viz: Build System
mv3: Bundle Size Stats
mv2: E2e Actions Stats
storybook: Storybook
typescript migration: Dashboard
all artifacts
bundle viz:
- background: 0, 1, 2, 3, 4, 5, 6, 7
- common: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13
- ui: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13
- content-script: 0
- offscreen: 0

Page Load Metrics (1539 ± 44 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	1382	1768	1538	86	41
		domContentLoaded	1377	1708	1521	81	39
		load	1381	1771	1539	91	44
		domInteractive	23	94	39	21	10
		backgroundConnect	8	66	21	18	8
		firstReactRender	14	68	29	20	10
		getState	4	47	9	12	6
		initialActions	0	1	0	0	0
		loadScripts	972	1287	1103	75	36
		setupStore	7	57	22	22	10
		uiStartup	1553	1964	1734	95	45

Bundle size diffs

background: 0 Bytes (0.00%)
ui: 0 Bytes (0.00%)
common: 0 Bytes (0.00%)

zone-live and others added 25 commits December 9, 2024 10:16

chore: controllers and selector

0f5ffe2

chore: ui display

2b86780

chore: adds activity list and code fences

38e51f3

chore: adds tests and lint fixes

e7174a4

Merge branch 'main' into SOL-46-adds-the-multichain-transactions-cont…

7a853df

…roller

Merge branch 'main' into SOL-46-adds-the-multichain-transactions-cont…

3a2c370

…roller

chore: clean yarn lock file

b6efd42

chore: clean up

1c86b76

Update LavaMoat policies

a6a92d6

chore: adds missing code fences

f79963d

chore: update with main

13a361c

Merge branch 'SOL-46-adds-the-multichain-transactions-controller' of …

181c5ee

…github.com:MetaMask/metamask-extension into SOL-46-adds-the-multichain-transactions-controller

chore: updates yarn lock

b1be837

chore: remove unused locale

6637880

chore: remove unnecessary assert

c8af9bc

chore: update

f567362

chore: remove unnecessary assert

a1f3d21

chore: adds the ability to filter by mainnet txs only

c465217

chore: lint fix

f39282a

chore: lint undo

e6543dc

Merge branch 'main' into SOL-46-adds-the-multichain-transactions-cont…

43a89fb

…roller

chore: adds tests

0fb59a4

chore: lint fix

128e4a6

Merge branch 'main' into SOL-46-adds-the-multichain-transactions-cont…

8bb949c

…roller

chore: reset to main the policies and yarn lock, fix prettier

08c396f

metamaskbot added the INVALID-PR-TEMPLATE PR's body doesn't match template label Dec 18, 2024

chore: lint fix

6ad4665

metamaskbot removed the INVALID-PR-TEMPLATE PR's body doesn't match template label Dec 18, 2024

Merge branch 'SOL-46-adds-the-multichain-transactions-controller' int…

915a0e9

…o SOL-80-transaction-details

zone-live marked this pull request as ready for review January 27, 2025 14:04

zone-live requested review from ccharly and aganglada January 27, 2025 14:05

aganglada previously approved these changes Jan 28, 2025

View reviewed changes

ccharly reviewed Jan 28, 2025

View reviewed changes

Update ui/components/app/multichain-transaction-details-modal/multich…

5d75926

…ain-transaction-details-modal.test.tsx Co-authored-by: Charly Chevalier <[email protected]>

zone-live dismissed aganglada’s stale review via 5d75926 January 28, 2025 16:09

Update ui/components/app/multichain-transaction-details-modal/multich…

d6e121f

…ain-transaction-details-modal.test.tsx Co-authored-by: Charly Chevalier <[email protected]>

zone-live added 3 commits January 29, 2025 13:00

chore: review points

e5367df

chore: lint fix

c0116e2

chore: clean up

28574ca

Merge branch 'main' into SOL-80-transaction-details

d6d7a16

zone-live and others added 4 commits January 29, 2025 15:41

chore: small update for solana timestaps

4be87de

Merge branch 'main' into SOL-80-transaction-details

14dd87f

chore: test

af5bb11

chore: update privacy snapshot

3403374

zone-live requested a review from a team as a code owner January 29, 2025 17:32

zone-live requested review from aganglada and ccharly January 29, 2025 17:52

aganglada previously approved these changes Jan 30, 2025

View reviewed changes

chore: reset file

5d0d442

zone-live dismissed aganglada’s stale review via 5d0d442 January 30, 2025 09:38

Merge branch 'main' into SOL-80-transaction-details

6996ee1

aganglada approved these changes Jan 30, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: SOL-80 transaction details #29323

feat: SOL-80 transaction details #29323

zone-live commented Dec 18, 2024 •

edited

Loading

github-actions bot commented Dec 18, 2024

metamaskbot commented Jan 28, 2025

metamaskbot commented Jan 29, 2025

metamaskbot commented Jan 29, 2025

zone-live commented Jan 29, 2025

metamaskbot commented Jan 29, 2025

metamaskbot commented Jan 30, 2025

feat: SOL-80 transaction details #29323

Are you sure you want to change the base?

feat: SOL-80 transaction details #29323

Conversation

zone-live commented Dec 18, 2024 • edited Loading

Description

Related issues

Manual testing steps

Screenshots/Recordings

Before

After

Pre-merge author checklist

Pre-merge reviewer checklist

github-actions bot commented Dec 18, 2024

metamaskbot commented Jan 28, 2025

metamaskbot commented Jan 29, 2025

metamaskbot commented Jan 29, 2025

zone-live commented Jan 29, 2025

metamaskbot commented Jan 29, 2025

metamaskbot commented Jan 30, 2025

zone-live commented Dec 18, 2024 •

edited

Loading