feat: Create UseScaffoldWatchContractEvent Hook #584
Conversation
metalboyrick
changed the title
packages/nextjs/hooks/scaffold-stark/useScaffoldWatchContractEvent.ts
Outdated
Show resolved
Hide resolved
packages/nextjs/hooks/scaffold-stark/useScaffoldWatchContractEvent.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Seems on my side I cannot watch events
https://www.loom.com/share/21e38e3d5c374186a3339cd354f0ce7e?sid=12287af6-510f-49e6-bc39-6286b6390002
Also might need you to update the documentation with the features you have put in, but let's resolve this feature to be working properly first
packages/nextjs/hooks/scaffold-stark/useScaffoldWatchContractEvent.ts
Outdated
Show resolved
Hide resolved
packages/nextjs/hooks/scaffold-stark/useScaffoldWatchContractEvent.ts
Outdated
Show resolved
Hide resolved
|
https://www.loom.com/share/c53f4244651a4c84addd4b09bfef8569?sid=dc6e0deb-f768-4f2f-8b18-a528b792079e @metalboyrick I'll remove the '.then()' and the commented code though |
There was a problem hiding this comment.
looks good to me,
I will merge this PR, in the mean time would like your assistance to update this
https://scaffoldstark.com/docs/hooks/useScaffoldWatchContractEvent
Repo: https://github.com/Scaffold-Stark/scaffoldstark-site
Implement useScaffoldWatchContractEvent
Fixes #569
Types of change
Comments (optional)
This PR implements the useScaffoldWatchContractEvent hook, just using the polling mechanism.
The webhook mechanism was not used because for it to work, it would have to import the Webhook-related parameters from context, and #555 is not merged yet.
When it has been resolved, the polling can be used.
This PR has been tested on the ch-0 branch of speedrunstark, and is confirmed to work, the maintainers can test it further