Skip to content

fix: profile wallets tab shown in wrong order#5378

Merged
0xmkh merged 3 commits intomainfrom
fix/ton-ui
Dec 1, 2025
Merged

fix: profile wallets tab shown in wrong order#5378
0xmkh merged 3 commits intomainfrom
fix/ton-ui

Conversation

@0xmkh
Copy link
Copy Markdown
Collaborator

@0xmkh 0xmkh commented Dec 1, 2025

Description

Tabs displayed in hardcoded order (EVM, Solana, Bitcoin, TON) instead of user's configured network order. Clicking tabs navigated to wrong networks.

Type of change

  • Chore (non-breaking change that addresses non-functional tasks, maintenance, or code quality improvements)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Associated Issues

For Linear issues: Closes APKT-4319
For GH issues: closes #5370

Showcase (Optional)

If there is a UI change include the screenshots with before and after state.
If new feature is being introduced, include the link to demo recording.

Checklist

  • Code in this PR is covered by automated tests (Unit tests, E2E tests)
  • My changes generate no new warnings
  • I have reviewed my own code
  • I have filled out all required sections
  • I have tested my changes on the preview link
  • Approver of this PR confirms that the changes are tested on the preview link

Note

Profile wallets tabs now render in the user-configured chain order instead of a hardcoded order, with tests added to validate ordering across EVM, Solana, Bitcoin, and TON.

  • Scaffold UI (w3m-profile-wallets-view):
    • Tabs ordering logic updated to derive availableTabs from this.namespaces order: this.namespaces.map(...).filter(Boolean) ensuring user-configured chain order.
    • Added unit tests to verify tab rendering and ordering across various chain configurations (including TON and Bitcoin) and to ensure no tabs when only one namespace exists.
  • Changeset:
    • Patch version bumps across multiple packages with note: fixed profile wallets tabs order.

Written by Cursor Bugbot for commit e08bb62. This will update automatically on new commits. Configure here.

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Dec 1, 2025

🦋 Changeset detected

Latest commit: e08bb62

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

This PR includes changesets to release 26 packages
Name Type
@reown/appkit-scaffold-ui Patch
pay-test-exchange Patch
@reown/appkit-adapter-bitcoin Patch
@reown/appkit-adapter-ethers Patch
@reown/appkit-adapter-ethers5 Patch
@reown/appkit-adapter-solana Patch
@reown/appkit-adapter-ton Patch
@reown/appkit-adapter-wagmi Patch
@reown/appkit Patch
@reown/appkit-utils Patch
@reown/appkit-cdn Patch
@reown/appkit-cli Patch
@reown/appkit-codemod Patch
@reown/appkit-common Patch
@reown/appkit-controllers Patch
@reown/appkit-core Patch
@reown/appkit-experimental Patch
@reown/appkit-pay Patch
@reown/appkit-polyfills Patch
@reown/appkit-siwe Patch
@reown/appkit-siwx Patch
@reown/appkit-testing Patch
@reown/appkit-ui Patch
@reown/appkit-universal-connector Patch
@reown/appkit-wallet Patch
@reown/appkit-wallet-button 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

@vercel
Copy link
Copy Markdown

vercel bot commented Dec 1, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
appkit-basic-html Ready Ready Preview Comment Dec 1, 2025 10:35am
appkit-demo Ready Ready Preview Comment Dec 1, 2025 10:35am
appkit-gallery Ready Ready Preview Comment Dec 1, 2025 10:35am
appkit-headless-sample-app Ready Ready Preview Comment Dec 1, 2025 10:35am
appkit-laboratory Ready Ready Preview Comment Dec 1, 2025 10:35am
10 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
appkit-basic-example Ignored Ignored Dec 1, 2025 10:35am
appkit-basic-sign-client-example Ignored Ignored Dec 1, 2025 10:35am
appkit-basic-up-example Ignored Ignored Dec 1, 2025 10:35am
appkit-ethers5-bera Ignored Ignored Dec 1, 2025 10:35am
appkit-nansen-demo Ignored Ignored Dec 1, 2025 10:35am
appkit-vue-solana Ignored Ignored Dec 1, 2025 10:35am
appkit-wagmi-cdn-example Ignored Ignored Dec 1, 2025 10:35am
ethereum-provider-wagmi-example Ignored Ignored Dec 1, 2025 10:35am
next-wagmi-solana-bitcoin-example Ignored Ignored Dec 1, 2025 10:35am
vue-wagmi-example Ignored Ignored Dec 1, 2025 10:35am

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 1, 2025

Visual Regression Test Results ✅ Passed

✨ No visual changes detected

Chromatic Build: https://www.chromatic.com/build?appId=6493191bf4b10fed8ca7353f&number=474
Storybook Preview: https://6493191bf4b10fed8ca7353f-znjwnhkchu.chromatic.com/

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 1, 2025

📦 Bundle Size Check

All bundles are within size limits

📊 View detailed bundle sizes

> @reown/appkit-monorepo@1.7.1 size /home/runner/work/appkit/appkit


> size-limit

@reown/appkit - Main Entry
Size limit:   80 kB
Size:         71.36 kB with all dependencies, minified and gzipped
Loading time: 1.4 s    on slow 3G
Running time: 374 ms   on Snapdragon 410
Total time:   1.8 s
@reown/appkit/react
Size limit:   230 kB
Size:         228.31 kB with all dependencies, minified and gzipped
Loading time: 4.5 s     on slow 3G
Running time: 662 ms    on Snapdragon 410
Total time:   5.2 s
@reown/appkit/vue
Size limit:   80 kB
Size:         71.36 kB with all dependencies, minified and gzipped
Loading time: 1.4 s    on slow 3G
Running time: 228 ms   on Snapdragon 410
Total time:   1.7 s
@reown/appkit-scaffold-ui
Size limit:   220 kB
Size:         209.4 kB with all dependencies, minified and gzipped
Loading time: 4.1 s    on slow 3G
Running time: 508 ms   on Snapdragon 410
Total time:   4.6 s
@reown/appkit-ui
Size limit:   500 kB
Size:         13.15 kB with all dependencies, minified and gzipped
Loading time: 257 ms   on slow 3G
Running time: 43 ms    on Snapdragon 410
Total time:   299 ms

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 1, 2025

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 79.71% 38341 / 48097
🔵 Statements 79.71% 38341 / 48097
🔵 Functions 77.3% 4107 / 5313
🔵 Branches 86.63% 9306 / 10742
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
packages/scaffold-ui/src/views/w3m-profile-wallets-view/index.ts 81.61% 85.03% 77.5% 81.61% 124, 128, 137-139, 145-149, 155-156, 180, 197, 236, 254, 273, 280-281, 324-326, 334, 348, 369, 402, 405, 424, 452, 487, 491-509, 524-526, 534-539, 542, 560-561, 565-571, 579-580, 583-588, 600-603, 608-609, 614-615, 665, 694, 710-711, 726-731, 737-742, 745-750
Generated in workflow #16433 for commit e08bb62 by the Vitest Coverage Report Action

Co-authored-by: glitch-txs <glitch-txs@users.noreply.github.com>
Comment thread packages/scaffold-ui/src/views/w3m-profile-wallets-view/index.ts Outdated
@linear
Copy link
Copy Markdown

linear bot commented Dec 1, 2025

@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 1, 2025
@0xmkh 0xmkh added this pull request to the merge queue Dec 1, 2025
Merged via the queue into main with commit 2867bfd Dec 1, 2025
45 checks passed
@0xmkh 0xmkh deleted the fix/ton-ui branch December 1, 2025 12:05
@github-actions github-actions bot locked and limited conversation to collaborators Dec 1, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Broken UI when using TON — Profile Wallets view swaps Solana & TON

2 participants