-
Notifications
You must be signed in to change notification settings - Fork 164
Fix 575 #590
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
Fix 575 #590
Conversation
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.
GM Thanks for the PR! Comments down below
On the PR:
- Please rebase your branch to the latest develop
- please put the appropriate title (e.g.
feat: implement...
) - Please re-read the issue and try to understand the intention behind the task, feel free to consult us if you have any questions
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 don't think you need to implement a contract
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 don't think you need to implement a contract
const multicallResult = useReadContract({ | ||
functionName: "call_contracts", | ||
address: multicallContract?.address, | ||
abi: multicallContract?.abi, | ||
watch: true, | ||
args: [contracts, entryPointSelectors, calldata], | ||
enabled: allContractsReady && contracts.length > 0, | ||
blockIdentifier: "pending" as BlockNumber, | ||
...readConfig, | ||
}); |
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 are not supposed to call a particular contract here, you have to use starknet.js
or starknet-react
to implement a hook that can read multiple contracts at once.
Implement use Scaffold multiread hook
Fixes #575
Types of change
Comments (optional)