Skip to content

docs: fix README examples and add useAccount documentation #128

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
GuiBibeau merged 2 commits into from
Jan 5, 2026
Merged

docs: fix README examples and add useAccount documentation #128

GuiBibeau merged 2 commits into from
Jan 5, 2026

Conversation

@GuiBibeau
Copy link
Collaborator

@GuiBibeau GuiBibeau commented Jan 5, 2026 •
edited
Loading

Summary

This PR bundles 4 documentation fixes for the @solana/react-hooks README:

  • Fix useSolTransfer parameter name: Changed lamports to amount in the example
  • Fix useSplToken bigint usage: Added amountInBaseUnits: true when passing bigint amounts
  • Document useWalletConnection return values: Added currentConnector to destructuring example with usage
  • Document useAccount hook: Added new section with usage example

Test plan

  • Verify README examples render correctly on GitHub
  • Verify the code snippets are syntactically valid

Fixes #113
Fixes #115
Fixes #108
Fixes #110

@GuiBibeau
docs: fix README examples and add useAccount documentation
9a3cd8e 
- Fix useSolTransfer: lamports → amount parameter (#113)
- Fix useSplToken: add amountInBaseUnits for bigint (#115)
- Add currentConnector to useWalletConnection example (#108)
- Document useAccount hook with example (#110)
@GuiBibeau
docs: add note explaining amountInBaseUnits parameter
6a9529a
@GuiBibeau GuiBibeau merged commit d2ca1ca into main Jan 5, 2026
7 checks passed
@github-actions github-actions bot mentioned this pull request Jan 5, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

2 participants

@GuiBibeau