BootNodeDev / dAppBooster

A modern blockchain boilerplate built to quickly get you started with your next project.
https://dappbooster.dev
MIT License
5 stars 1 forks source link

fix: useL1CrossDomainMessengerProxy changing hooks by functions and only called when needed #302

Closed nicosampler closed 1 month ago

nicosampler commented 1 month ago

Fixes a bug when consuming useL1CrossDomainMessengerProxy hook. Now it is avoided to call the L1 gas estimation and L2 data info when the hook is initalized, these information is called when the send tx is called.

Description:

(Please include a summary of the changes and the related issue)

Steps:

(Required steps to reproduce or test the fix / feature)

Type of change:

How Has This Been Tested?

Remember to check that:

Screenshots

(Add screenshots or videos to help test this pull request)

vercel[bot] commented 1 month ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
dappbooster ✅ Ready (Inspect) Visit Preview Sep 17, 2024 2:08pm
docs.dappbooster ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 17, 2024 2:08pm
nicosampler commented 1 month ago

@nicosampler we may be missing a big opportunity here to add the op-stack, wdyt?

basically, this was my idea, what else do you have in mind?