-
Notifications
You must be signed in to change notification settings - Fork 199
feat: solana demo #1583
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.
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
feat: solana demo #1583
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
🌿 Documentation Preview
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good! just a couple of styling/UX questions.
| setTransactionState("signing"); | ||
| setTransactionState("sponsoring"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems weird to me that we go from signing to sponsoring w/o anything happening in-between?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nvm, i see @blakecduncan already commented this on #1578 .
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I pinged with margaret about this earlier, I agree.
21ee97e to
f4c00cc
Compare
f4c00cc to
014f4ab
Compare
blakecduncan
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯
#1578
Re do of
Pull Request Checklist
yarn test)sitefolder, and guidelines for updating/adding docs can be found in the contribution guide)feat!: breaking change)yarn lint:check) and fix any issues? (yarn lint:write)PR-Codex overview
This PR primarily focuses on updating the Solana integration in the project. It includes changes to connection handling, transaction creation, and the introduction of a new NFT card component.
Detailed summary
duckImage.png.policyIdinconfig.tsxto useNEXT_PUBLIC_SOLANA_POLICY_ID.useSolanaConnectionhook inuseSolanaConnection.ts.SolanaCardtoSolanaNftCard.Stagecomponent inStage.tsx.useSolanaTransaction.ts.SolanaNftCard.tsx.