Skip to content

Conversation

@Blu-J
Copy link
Contributor

@Blu-J Blu-J commented May 2, 2025

#1578

Re do of

Pull Request Checklist


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

  • Deleted duckImage.png.
  • Updated policyId in config.tsx to use NEXT_PUBLIC_SOLANA_POLICY_ID.
  • Introduced useSolanaConnection hook in useSolanaConnection.ts.
  • Renamed SolanaCard to SolanaNftCard.
  • Added Stage component in Stage.tsx.
  • Modified transaction handling in useSolanaTransaction.ts.
  • Implemented NFT minting logic in SolanaNftCard.tsx.

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

Jade Hamilton added 2 commits May 2, 2025 10:08
@vercel
Copy link

vercel bot commented May 2, 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 May 8, 2025 4:19pm
aa-sdk-ui-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 8, 2025 4:19pm

@github-actions
Copy link

github-actions bot commented May 2, 2025

🌿 Documentation Preview

Name Status Preview Updated (UTC)
Alchemy Docs ✅ Ready 🔗 Visit Preview May 8, 2025, 4:17 PM

@github-actions github-actions bot temporarily deployed to docs-preview May 2, 2025 16:34 Inactive
Copy link
Contributor

@jakehobbs jakehobbs left a 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.

Comment on lines +47 to +48
setTransactionState("signing");
setTransactionState("sponsoring");
Copy link
Contributor

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?

Copy link
Contributor

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 .

Copy link
Contributor Author

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.

@Blu-J Blu-J force-pushed the bluj/re-do-solana-demo branch from f4c00cc to 014f4ab Compare May 2, 2025 17:33
@github-actions github-actions bot temporarily deployed to docs-preview May 2, 2025 17:33 Inactive
@Blu-J Blu-J changed the title Bluj/re do solana demo feat: solana demo May 2, 2025
@Blu-J Blu-J changed the title feat: solana demo feat: solana demo re revert May 2, 2025
@github-actions github-actions bot temporarily deployed to docs-preview May 2, 2025 19:42 Inactive
@github-actions github-actions bot temporarily deployed to docs-preview May 7, 2025 14:26 Inactive
@Blu-J Blu-J changed the title feat: solana demo re revert feat: solana demo May 7, 2025
Copy link
Collaborator

@blakecduncan blakecduncan left a comment

Choose a reason for hiding this comment

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

💯

@Blu-J Blu-J merged commit d471359 into main May 8, 2025
8 checks passed
@Blu-J Blu-J deleted the bluj/re-do-solana-demo branch May 8, 2025 16:57
jakehobbs added a commit that referenced this pull request May 8, 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.

4 participants