Closed nicosampler closed 3 months ago
Latest commit: ee2fa0abaa937b8c91bac9cf1f8862b18f3dc56f
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
CI is running/has finished running commands for commit ee2fa0abaa937b8c91bac9cf1f8862b18f3dc56f. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.
📂 See all runs for this CI Pipeline Execution
Sent with 💌 from NxCloud.
The
useCannonRegistry
hook is utilized byuseCannonBuild
anduseCannonPackage
, which are hooks used in many components. Previously, each component that consumed one of these hooks would trigger the creation of provider instances for all chains. This refactor ensures that these instances are created only once when the dApp is initialized.Ideally, we should identify all top-level components using the hook and register the provider at the highest appropriate level. However, given the widespread use of these hooks, defining it on the
_app.ts
file seems temporarily a valid option.