fix: handle BigInt serialization for ecosystem wallets

[joaquim-verges]

Problem solved

Short description of the bug fixed or feature added

---

PR-Codex overview

The focus of this PR is to handle biging serialization for ecosystem wallets.

Detailed summary

• Updated signTypedData, signMessage, and signTransaction functions to use stringify method for serialization
• Improved handling of ecosystem wallets in the in-app/web module

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
docs-v2	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Sep 23, 2024 5:33am
thirdweb_playground	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Sep 23, 2024 5:33am
thirdweb-www	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Sep 23, 2024 5:33am
wallet-ui	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Sep 23, 2024 5:33am

[changeset-bot]

🦋 Changeset detected

Latest commit: 33936b3

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
thirdweb	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[graphite-app]

Your org has enabled the Graphite merge queue for merging into main

Add the label “merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

[joaquim-verges]

• fix: handle BigInt serialization for ecosystem wallets #4756 👈
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @joaquim-verges and the rest of your teammates on Graphite

[github-actions]

size-limit report 📦

Path	Size	Loading time (3g)	Running time (snapdragon)	Total time
thirdweb (esm)	41.89 KB (0%)	838 ms (0%)	2.7 s (-15.33% 🔽)	3.5 s
thirdweb (cjs)	101.38 KB (0%)	2.1 s (0%)	5.7 s (-15.81% 🔽)	7.8 s
thirdweb (minimal + tree-shaking)	4.82 KB (0%)	97 ms (0%)	277 ms (-35.75% 🔽)	374 ms
thirdweb/chains (tree-shaking)	498 B (0%)	10 ms (0%)	357 ms (+41.21% 🔺)	367 ms
thirdweb/react (minimal + tree-shaking)	16.73 KB (0%)	335 ms (0%)	694 ms (-28.04% 🔽)	1.1 s

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 49.06%. Comparing base (1133ccc) to head (33936b3).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4756      +/-   ##
==========================================
- Coverage   49.06%   49.06%   -0.01%     
==========================================
  Files        1054     1054              
  Lines       56860    56860              
  Branches     3888     3890       +2     
==========================================
- Hits        27900    27899       -1     
- Misses      28327    28328       +1     
  Partials      633      633              

Flag	Coverage Δ		*Carryforward flag
legacy_packages	65.68% <ø> (ø)		Carriedforward from 1133ccc
packages	45.13% <ø> (-0.01%)	⬇️

*This pull request uses carry forward flags. Click here to find out more.

see 1 file with indirect coverage changes

[graphite-app]

Merge activity

• Sep 23, 1:28 AM EDT: joaquim-verges added this pull request to the Graphite merge queue.
• Sep 23, 1:33 AM EDT: joaquim-verges merged this pull request with the Graphite merge queue.