-
Notifications
You must be signed in to change notification settings - Fork 323
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
Content for Frontend Setup in Base Camp #162
Conversation
apps/base-docs/base-camp/docs/frontend-setup/building-an-onchain-app.md
Outdated
Show resolved
Hide resolved
apps/base-docs/base-camp/docs/frontend-setup/building-an-onchain-app.md
Outdated
Show resolved
Hide resolved
Review Error for amhayslipcb @ 2023-11-28 18:08:49 UTC |
Approved review 1769608705 from taycaldwell is now dismissed due to new commit. Re-request for approval.
apps/base-docs/base-camp/docs/reading-and-displaying-data/configuring-useContractRead.md
Outdated
Show resolved
Hide resolved
apps/base-docs/base-camp/docs/reading-and-displaying-data/useAccount.md
Outdated
Show resolved
Hide resolved
apps/base-docs/base-camp/docs/reading-and-displaying-data/useAccount.md
Outdated
Show resolved
Hide resolved
apps/base-docs/base-camp/docs/reading-and-displaying-data/useContractRead.md
Show resolved
Hide resolved
|
||
:::caution | ||
|
||
If you're using your own contract, please redeploy it with the following `view` functions: |
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.
I got lost here, how do I redeploy it?
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.
We're assuming the learner knows how at this point, and earlier up we linked to the original exercise. We've also got a copy of a contract they can use down below!
|
||
To start, you'll need to put some data into your contract so that you can read it from your frontend. Open [Goerli BaseScan], find your contract, connect with your wallet, and call the `claim` function. | ||
|
||
Add the following two issues: |
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.
I was asking how do you redeploy it because would it make sense to add the issues during the redeployment stage? or we want to also teach interacting with the contract via BaseScan? But it means the contract has to be verified? Sorry, probably I'm misunderstanding something
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.
We're just using it temporarily to send transactions so we have something to read. Seemed like this was the simplest way.
Review Error for CryptoUGuy @ 2023-12-11 21:52:17 UTC |
* Stashing progress * Finalize building an app article * Address feedback * Fix typo * Add useAccount step by step * Add useContractRead step by step * Add configuring useContractRead * Finish intro for configuring useContractRead * Add useContractWrite * Add usePrepareContractWrite * Respond to feedback * Sidebar and final edits
What changed? Why?
New content
Notes to reviewers
How has it been tested?