fix: mm connect flow #198

itaven · 2024-11-04T13:06:54Z

updated mm connectWallet() flow
updated locator/flow for confirmTransactionOfTokenApproval()
added viem for wallets package to get getCheckSum address 0x107209ce35ab1a0b3a22593ba85382817f471cd -> 0x107209CE35Ab1a0B3a22593ba85382817F471CDe since MM shows only in lower case. Bumped ts version to resolve conflicts with viem.
- MM provide checkSummed address but only with "Copy to clipboard" button but for wallets setup there is no option to get from clippoard since metamask/lavamoat blocked it.

packages/wallets/src/metamask/metamask.page.ts

itaven added 2 commits November 4, 2024 18:06

fix: mm connect flow

d047f41

fix: set update-mm branch as alpha

038e8a4

itaven had a problem deploying to develop November 4, 2024 13:24 — with GitHub Actions Failure

itaven added 4 commits November 5, 2024 15:40

fix: wrap flow

01b7050

fix: getWalletAddress()

9ae2a7e

fix: up ts && add viem

a2b4e87

fix: remove unused var

c56b0b0

itaven had a problem deploying to develop November 5, 2024 10:54 — with GitHub Actions Failure

jake4take reviewed Nov 5, 2024

View reviewed changes

packages/wallets/src/metamask/metamask.page.ts Outdated Show resolved Hide resolved

itaven added 2 commits November 5, 2024 16:08

fix: use const accountDetailAddressLabel from popoverElements

481211a

fix: unmock release branch

39d0d68

jake4take previously approved these changes Nov 5, 2024

View reviewed changes

fix: remove unused step for confirmTransactionOfTokenApproval()

df7f78a

itaven dismissed jake4take’s stale review via df7f78a November 5, 2024 11:59

itaven temporarily deployed to develop November 5, 2024 12:00 — with GitHub Actions Inactive

Vorobeyko approved these changes Nov 5, 2024

View reviewed changes

itaven deployed to develop November 5, 2024 12:02 — with GitHub Actions Active

itaven merged commit 8273ca7 into main Nov 5, 2024
6 checks passed

itaven deleted the update-mm branch November 5, 2024 12:05

Provide feedback