Closed sideninja closed 1 month ago
The recent changes focus on refactoring how storage addresses are handled within the application. The hardcoded previewnetStorageAddress
variable has been removed from the main implementation, allowing for a more dynamic approach to retrieving storage addresses based on the current network configuration. This shift enhances flexibility and maintainability, particularly in test scenarios and when adapting to different environments.
Files | Change Summary |
---|---|
services/requester/remote_state.go , services/requester/requester.go |
Removed previewnetStorageAddress from the main implementation, replacing it with dynamic retrieval using evm.StorageAccountAddress . |
services/requester/remote_state_test.go |
Added imports for evm and flowGo ; introduced previewnetStorageAddress initialized with the new storage address for testing. |
🐰 In code we hop, with changes bright,
No more hard paths, just pure delight.
Dynamic flows like rivers run,
A simpler way, oh what fun!
With every tweak, we leap and cheer,
For cleaner code that brings us near! 🌟
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Fix storage address used for remote ledger to be derived from chain ID
For contributor use:
master
branchFiles changed
in the Github PR explorerSummary by CodeRabbit
New Features
Bug Fixes
Documentation