Skip to content

Conversation

@joaquim-verges
Copy link
Member

@joaquim-verges joaquim-verges commented Feb 17, 2025

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.

Releases

[email protected]

Patch Changes

  • #6274 b182302 Thanks @kumaryash90! - bytes32 salt for deterministic deployment

  • #6283 5a08176 Thanks @joaquim-verges! - Respect supportedTokens override in PayEmbed

  • #6281 a0f3557 Thanks @joaquim-verges! - Fix text wrapping for long balances in token selector

  • #6282 45ca033 Thanks @joaquim-verges! - Deprecated viemAdapter.walletClient in favor of viemAdapter.wallet to take wallet instances instead of accounts

    BEFORE:

    import { viemAdapter } from "thirdweb/adapters/viem";
    
    const walletClient = viemAdapter.walletClient.toViem({
      account, // Account
      chain,
      client,
    });

    AFTER:

    import { viemAdapter } from "thirdweb/adapters/viem";
    
    const walletClient = viemAdapter.wallet.toViem({
      wallet, // now pass a connected Wallet instance instead of an account
      chain,
      client,
    });

    This allows for full wallet lifecycle management with the viem adapter, including switching chains, adding chains, events and more.

@thirdweb-dev/[email protected]

[email protected]

Patch Changes


PR-Codex overview

This PR primarily focuses on updating package versions and changelogs for various components of the thirdweb project, particularly enhancing the thirdweb-login and wagmi-adapter packages. It also includes updates to dependencies and improvements in wallet management.

Detailed summary

  • Updated version in apps/login/package.json from 0.1.8 to 0.1.9
  • Updated version in packages/thirdweb/package.json from 5.88.5 to 5.88.6
  • Updated version in packages/wagmi-adapter/package.json from 0.2.19 to 0.2.20
  • Added changelog entries for apps/login and packages/thirdweb with new features and fixes
  • Changed viemAdapter.walletClient to viemAdapter.wallet for improved wallet instance management

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

@joaquim-verges joaquim-verges requested review from a team as code owners February 17, 2025 21:51
@vercel
Copy link

vercel bot commented Feb 17, 2025

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 Feb 18, 2025 5:14am
login ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 18, 2025 5:14am
thirdweb_playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 18, 2025 5:14am
thirdweb-www ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 18, 2025 5:14am
wallet-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 18, 2025 5:14am

@github-actions github-actions bot added packages SDK Involves changes to the thirdweb SDK labels Feb 17, 2025
@graphite-app
Copy link
Contributor

graphite-app bot commented Feb 17, 2025

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • merge-queue - adds this PR to the back of the merge queue
  • hotfix - for urgent hot fixes, skip the queue and merge this PR next

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

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 17, 2025

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
thirdweb (esm) 46.3 KB (0%) 926 ms (0%) 3.4 s (+97.79% 🔺) 4.3 s
thirdweb (cjs) 121.89 KB (0%) 2.5 s (0%) 4.4 s (+3.76% 🔺) 6.8 s
thirdweb (minimal + tree-shaking) 5.6 KB (0%) 112 ms (0%) 275 ms (+35.39% 🔺) 387 ms
thirdweb/chains (tree-shaking) 506 B (0%) 10 ms (0%) 74 ms (+10.18% 🔺) 84 ms
thirdweb/react (minimal + tree-shaking) 19.29 KB (0%) 386 ms (0%) 339 ms (+21.33% 🔺) 725 ms

@codecov
Copy link

codecov bot commented Feb 17, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.83%. Comparing base (b182302) to head (d824801).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6280      +/-   ##
==========================================
- Coverage   56.85%   56.83%   -0.03%     
==========================================
  Files        1168     1168              
  Lines       64677    64793     +116     
  Branches     5246     5259      +13     
==========================================
+ Hits        36772    36824      +52     
- Misses      27177    27241      +64     
  Partials      728      728              
Flag Coverage Δ *Carryforward flag
legacy_packages 65.68% <ø> (ø) Carriedforward from b182302
packages 55.04% <ø> (-0.03%) ⬇️

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

see 4 files with indirect coverage changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

packages SDK Involves changes to the thirdweb SDK

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants