Closed beer-1 closed 1 month ago
[!CAUTION]
Review failed
The pull request is closed.
The updates involve modifying the visibility of the sudo_transfer
function in both the initia_std::primary_fungible_store
and minitia_std::primary_fungible_store
modules. The changes remove the entry
designation, enhancing the function's accessibility while maintaining its friend
visibility. This adjustment simplifies interactions with the function from other modules, allowing for greater flexibility without impacting its core functionality.
Files | Change Summary |
---|---|
precompile/modules/initia_stdlib/sources/fa/primary_fungible_store.move precompile/modules/minitia_stdlib/sources/fa/primary_fungible_store.move |
The sudo_transfer function's visibility changed from public(friend) entry to public(friend) , allowing easier access while retaining friend access control. |
sequenceDiagram
participant ModuleA as initia_std::primary_fungible_store
participant ModuleB as minitia_std::primary_fungible_store
participant External as External Module
External->>ModuleA: Call sudo_transfer()
ModuleA-->>External: Transfer successful
External->>ModuleB: Call sudo_transfer()
ModuleB-->>External: Transfer successful
🐇 In the meadow where changes bloom,
Thesudo_transfer
sheds its room.
No longer entry, but still a friend,
With simpler paths, the joy won't end!
Hopping forth with glee and cheer,
More access granted, our path is clear! 🌼
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?
Thanks to Zellic for reporting!
Summary by CodeRabbit
sudo_transfer
function in bothinitia_std
andminitia_std
modules, allowing for more straightforward interactions within the public interface.sudo_transfer
function, improving its usability and flexibility in different contexts.