-
Notifications
You must be signed in to change notification settings - Fork 266
docs: updat frontend guide to show new libraries #968
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
base: main
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
Replace @solana/kit with the newer, more convenient @solana/react-hooks package
fdb09ba to
1ec9b00
Compare
| } | ||
| ``` | ||
|
|
||
| ## !!steps 5. Run the Application |
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.
need a step before running app to update layout.tsx to use the provider
| import { useState } from "react"; | ||
| import { useSolTransfer, useWallet } from "@solana/react-hooks"; | ||
|
|
||
| const LAMPORTS_PER_SOL = 1_000_000_000n; |
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.
getting BigInt literals are not available when targeting lower than ES2020. lint when creating app from npx create-next-app@latest
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.
as an error or warning? I'm surprise they still show up that warning es2020 is 6 years ago soon
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.
|
General question, do you want to include all these in as a "Frontend" subsection or have a dedicated secondary nav section for all these packages? These pages are pretty light right now, but not sure if you're going to need more pages to fully document the packages in the future. You can also add the new packages on these pages: |
|
@ZYJLiu Thanks for the review, most was addressed. I implement it in templates and once there's good traction in downloads and more feedback i would do a follow up PR. I also want to replace all tutorials that use front-end to use this. I want to dog food gradually to also fix issues as we find them |

Updates frontend docs to use the newer
@solana/react-hookspackage.Part of a bigger revamp of frontend documentation.