Skip to content

Conversation

@Blu-J
Copy link
Contributor

@Blu-J Blu-J commented Apr 22, 2025

Pull Request Checklist


PR-Codex overview

This PR introduces support for Solana actions in the @account-kit/react library, enhancing the functionality for handling Solana transactions and integrating them into the existing account kit structure.

Detailed summary

  • Added "Solana Actions" section with hooks: useSolanaTransaction and useSolanaSignMessage.
  • Updated documentation for useSolanaTransaction with examples and parameters.
  • Enhanced AlchemySolanaWeb3Context with detailed usage instructions.
  • Introduced SolanaSigner class with methods for signing and transaction handling.
  • Updated sidebar references to include new Solana-related items.

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

@vercel
Copy link

vercel bot commented Apr 22, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
aa-sdk-site ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 23, 2025 9:42am
aa-sdk-ui-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 23, 2025 9:42am

@graphite-app
Copy link
Contributor

graphite-app bot commented Apr 22, 2025

How to use the Graphite Merge Queue

Add the label graphite-merge-queue to this PR to add it to the merge queue.

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

github-actions bot commented Apr 22, 2025

🌿 Documentation Preview

Name Status Preview Updated (UTC)
Alchemy Docs ❌ Failed Apr 23, 2025, 9:37 AM

@github-actions github-actions bot had a problem deploying to docs-preview April 22, 2025 21:54 Failure
* This hook is used to create a SolanaSigner instance.
* It is used to sign transactions and messages for the Solana blockchain.
*
* ```tsx
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding some docs

toAddress: "<toAddress>",
},
* ```
function MyComponent() {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wanted to have something that had the types, and was checked

}
```

## Other examples
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We probably should remove this, I didn't know where to put, then I found that the components where already being rendered

@github-actions github-actions bot had a problem deploying to docs-preview April 22, 2025 21:55 Failure
"useClientActions",
"useChain",
],
"Solana Actions": ["useSolanaTransaction", "useSolanaSignMessage"],
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was the biggest aha moment

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@moldy530 What about the provider? Should I make a context or make it part of the other alchemy provider, and should I add this in as a seperate config field, or part of chain setup, which I have to filter now I think for viem vs not

- page: useSolanaTransaction
path: wallets/pages/reference/account-kit/react/hooks/useSolanaTransaction.mdx
- page: useSolanaSigner
path: wallets/pages/reference/account-kit/react/hooks/useSolanaSigner.mdx
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this be useSolanaSignMessage.mdx instead of useSolanaSigner.mdx?

@Blu-J Blu-J closed this Apr 25, 2025
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.

2 participants