Skip to content
This repository was archived by the owner on Jan 20, 2026. It is now read-only.

SeiDB State Store ApplyChangeset Version 0#53

Merged
Kbhat1 merged 5 commits intomainfrom
ApplyChangesetUpdate
Jan 23, 2024
Merged

SeiDB State Store ApplyChangeset Version 0#53
Kbhat1 merged 5 commits intomainfrom
ApplyChangesetUpdate

Conversation

@Kbhat1
Copy link
Contributor

@Kbhat1 Kbhat1 commented Jan 23, 2024

Describe your changes and provide context

  • For ApplyChangeset, if version received is 0, apply changeset at version 1
  • Fixes issues with querying keys written at genesis (version 0)

Testing performed to validate your change

  • Added unit tests
  • Currently testing on node

@Kbhat1 Kbhat1 requested a review from yzang2019 January 23, 2024 17:17
Copy link
Contributor

@alexanderbez alexanderbez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ConceptACK

Copy link
Collaborator

@yzang2019 yzang2019 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we add an integration test for this as well?

@Kbhat1
Copy link
Contributor Author

Kbhat1 commented Jan 23, 2024

Can we add an integration test for this as well?

@yzang2019 integration tests are being added here sei-protocol/sei-chain#1242 . Will add another query to admin account (to repro genesis issues)

@Kbhat1 Kbhat1 merged commit 39eb03f into main Jan 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants