-
Notifications
You must be signed in to change notification settings - Fork 327
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
chore: Fix readContract docs #4920
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
Your org has enabled the Graphite merge queue for merging into mainAdd the label “merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix. You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link. |
packages/thirdweb/src/react/core/hooks/contract/useReadContract.ts
Outdated
Show resolved
Hide resolved
size-limit report 📦
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4920 +/- ##
=======================================
Coverage 47.61% 47.61%
=======================================
Files 1057 1057
Lines 57241 57241
Branches 3926 3925 -1
=======================================
Hits 27256 27256
Misses 29296 29296
Partials 689 689
*This pull request uses carry forward flags. Click here to find out more.
|
@@ -111,7 +111,7 @@ export type ReadContractOptions< | |||
* | |||
* const { data, isLoading } = useReadContract({ | |||
* contract, | |||
* method: "function tokenURI(uint256 tokenId) returns (string)"}), | |||
* method: "function tokenURI(uint256 tokenId) returns (string)"), |
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.
same here
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.
good bot :D
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.
Ser. Still wrong here
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.
hmm? i updated the code ser
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.
There's still an extra ')' in there at the end no?
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 think the bot comment got us confused. I did update this part, can no longer see the extra ")" in the code
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.
the comment should resolved on itself, but since we are using this thread maybe thats why it did
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.
Oh yeah that must be it. Looks fine on gh
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.
good bot
packages/thirdweb/src/react/core/hooks/contract/useReadContract.ts
Outdated
Show resolved
Hide resolved
Merge activity
|
## Problem solved Short description of the bug fixed or feature added <!-- start pr-codex --> --- ## PR-Codex overview This PR focuses on updating the `useReadContract` function calls in two files by adding a comma to the method definition for better syntax clarity. ### Detailed summary - In `packages/thirdweb/src/transaction/read-contract.ts`: - Added a comma after the method definition for `tokenURI`. - In `packages/thirdweb/src/react/core/hooks/contract/useReadContract.ts`: - Added a comma after the method definition for `tokenURI`. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->
6bd8f46
to
5c6f288
Compare
Problem solved
Short description of the bug fixed or feature added
PR-Codex overview
This PR updates the
useReadContract
function calls in two files to properly format themethod
string by adding a trailing comma. This change ensures consistency and correctness in the function's parameters.Detailed summary
In
packages/thirdweb/src/transaction/read-contract.ts
:method
string inuseReadContract
.In
packages/thirdweb/src/react/core/hooks/contract/useReadContract.ts
:method
string inuseReadContract
.