Skip to content

fix(mobile): remove dead useWallet.tsx duplicate hook and add shape t… - #1417

Open
kodinaka30-ship-it wants to merge 1 commit into
Epta-Node:mainfrom
kodinaka30-ship-it:fix/1304-usewallet-duplicate
Open

fix(mobile): remove dead useWallet.tsx duplicate hook and add shape t…#1417
kodinaka30-ship-it wants to merge 1 commit into
Epta-Node:mainfrom
kodinaka30-ship-it:fix/1304-usewallet-duplicate

Conversation

@kodinaka30-ship-it

Copy link
Copy Markdown

Summary

Removed the duplicate useWallet implementation and retained a single canonical hook path. Updated affected imports and added a shape test to prevent future ambiguity between wallet hook implementations.

Type of Change

  • Bug fix
  • New feature
  • Contract change (logic, storage, or API)
  • Documentation update
  • Refactor / chore

Testing Done

  • cargo test passes
  • New tests added for changed behaviour
  • Manually verified on Testnet (if applicable)
  • Database migration tested (if applicable)
  • TypeScript SDK tests passed (if applicable)

Checklist

  • Changes are focused — one concern per PR
  • If a contract function was added or changed, the README API table is updated
  • No unresolved merge conflicts
  • No secrets or private keys committed

Related Issue

Closes #1304

@drips-wave

drips-wave Bot commented Aug 29, 2026

Copy link
Copy Markdown

@kodinaka30-ship-it Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@vercel

vercel Bot commented Aug 29, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
linkora-social-web Skipped Skipped Aug 29, 2026 7:26am

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Mobile: two different useWallet hooks export from the same path with different return shapes (dead duplicate)

1 participant