Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

migrate rawStorage view to master #191

Open
ixje opened this issue May 10, 2023 · 2 comments
Open

migrate rawStorage view to master #191

ixje opened this issue May 10, 2023 · 2 comments

Comments

@ixje
Copy link

ixje commented May 10, 2023

-edit-
Adding details to the issue as this was previously discussed with harry in DM.

When debugging a contract with storage, the debugger will show a list of random numbers (these are 32-bit hashes of the storage key). See the first 15 seconds of this video where Harry explains it; https://youtu.be/HSYeXzUp65A?feature=shared&t=137

This issue is about reverting those key hash changes and turning it back into a "raw storage" view. Imagine you have a contract with 20 storage records. Then it is a real pain to find which random number in that list matches your storage key. Instead the list should be based on the Key value. Apparently it is part of the storage schema changes that are in the develop branch.

To quote what Harry said

I should be able to bring that over to the main branch w/o the rest of the storage schema work

@ixje
Copy link
Author

ixje commented Jul 29, 2024

@chenzhitong I think this is the most important (non bug) issue that should change to improve usability.

@ixje
Copy link
Author

ixje commented Jul 29, 2024

Basically we want what is called "raw storage" in this video: https://youtu.be/HSYeXzUp65A?feature=shared&t=1182

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant