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

Refactor creating new StorableSlab #324

Merged
merged 3 commits into from
Jul 6, 2023
Merged

Refactor creating new StorableSlab #324

merged 3 commits into from
Jul 6, 2023

Commits on Jul 6, 2023

  1. Refactor creating new StorableSlab 

    Currently, in order to create StorableSlab, user needs to:
- generate SlabID from storage
- create new StorableSlab
- store created StorableSlab in storage

This commit unexports StorableSlab fields and adds NewStorableSlab().
With this change, user can simply call NewStorableSlab().
    @fxamacker
    fxamacker committed Jul 6, 2023
    Configuration menu
    Copy the full SHA
    0c5a9e0 View commit details
    Browse the repository at this point in the history

  2. Rename ID to ValueID 

    ValueID identifies atree Array and OrderedMap while SlabID identifies
slab in storage.  SlabID should only be used to retrieve, store, and
remove slab in storage.
    @fxamacker
    fxamacker committed Jul 6, 2023
    Configuration menu
    Copy the full SHA
    3fa18fe View commit details
    Browse the repository at this point in the history

  3. Merge pull request #325 from onflow/fxamacker/rename-id-to-valueid 

    Rename ID to ValueID
    @fxamacker
    fxamacker authored Jul 6, 2023
    Configuration menu
    Copy the full SHA
    37f2201 View commit details
    Browse the repository at this point in the history