Skip to content

fix: include account in bitcoin getAccountAddresses#5610

Draft
ShaoyeLi wants to merge 1 commit intoreown-com:mainfrom
ShaoyeLi:codex/get-account-addresses-5590
Draft

fix: include account in bitcoin getAccountAddresses#5610
ShaoyeLi wants to merge 1 commit intoreown-com:mainfrom
ShaoyeLi:codex/get-account-addresses-5590

Conversation

@ShaoyeLi
Copy link
Copy Markdown

Summary

  • include the connected account when sending WalletConnect getAccountAddresses requests
  • correct the internal request typing to use structured address responses instead of undefined params
  • map returned address metadata into the bitcoin connector shape and add regression coverage for the request payload

Testing

  • Verification in this worktree was blocked by repeated npm registry connection resets during pnpm install --frozen-lockfile
  • Code changes were kept to the connector implementation, its focused test file, and a changeset

@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 30, 2026

@ShaoyeLi is attempting to deploy a commit to the Reown Team on Vercel.

A member of the Team first needs to authorize it.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Mar 30, 2026

🦋 Changeset detected

Latest commit: 465e652

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

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

@github-actions
Copy link
Copy Markdown
Contributor

Thank you for your contribution! We ask that you please read and sign our CTA Document before we can accept your contribution. You can sign the CTA simply by posting a Pull Request Comment with the following text:


I have read the CTA Document and I hereby sign the CTA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

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.

1 participant