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.

Sign up for GitHub

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

Jump to bottom

Use Vec<u8> as a value for the ContractsState table #655

Closed
2 tasks done
xgreenx opened this issue Jan 11, 2024 · 0 comments
Closed
2 tasks done

Use Vec<u8> as a value for the ContractsState table #655

xgreenx opened this issue Jan 11, 2024 · 0 comments
Assignees
@bvrooman

Comments

@xgreenx
Copy link
Collaborator

xgreenx commented Jan 11, 2024 •
edited
Loading

It is a preparation for the smooth update to support dynamic storage.

Required changes:
@xgreenx xgreenx mentioned this issue Jan 29, 2024
Closed
17 tasks
@bvrooman bvrooman self-assigned this Feb 1, 2024
@bvrooman bvrooman mentioned this issue Feb 6, 2024
Merged
This was referenced Feb 22, 2024
Merged
Merged
bvrooman pushed a commit to FuelLabs/fuel-core that referenced this issue Feb 29, 2024
@xgreenx
feat: Impl StorageRead and StorageWrite (#1713)
d54d957 
Related issues:
- FuelLabs/fuel-vm#655

This PR removes the specific implementation of `StorageRead` for
`ContractsRawCode`, and replaces it with a generic implementation over
`Blueprint` tables and `StructuredStorage`. This means that other
implementations of `TableWithBlueprint` will automatically receive this
implementation, including `ContractsStorage`, which uses the `Sparse`
Blueprint.

---------

Co-authored-by: xgreenx <xgreenx9999@gmail.com>
@xgreenx xgreenx closed this as completed Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bvrooman bvrooman

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Dentosal @bvrooman @xgreenx