Suggestion for snap_manageState Method in MetaMask #2217
AMIRKHANEF
started this conversation in
Ideas
Replies: 1 comment 1 reply
-
Thank you for the suggestion! Right now. |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Hello MetaMask Developers,
I'm currently working on developing a snap, and I came across the
snap_manageState
method. After reviewing the code, I have a suggestion regarding theManageStateResult
type.In the current implementation:
For the update operation, the result is null. I propose changing this to a boolean type, where true indicates a successful operation, and false indicates a failure.
What are your thoughts on such a change? Do you see potential benefits or drawbacks?
I appreciate your hard work on MetaMask and the support you provide to the developer community. Thank you for considering this suggestion!
Beta Was this translation helpful? Give feedback.
All reactions