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

Tx/VP prefix iter should include keys from write log #165

Closed
tzemanovic opened this issue Jun 24, 2021 · 2 comments · Fixed by #913
Closed

Tx/VP prefix iter should include keys from write log #165

tzemanovic opened this issue Jun 24, 2021 · 2 comments · Fixed by #913
Assignees
Labels
bug Something isn't working ledger prio:high

Comments

@tzemanovic
Copy link
Member

tzemanovic commented Jun 24, 2021

The prefix iterator for Tx and VP posterior state should also iterate over the storage keys from write log. Currently, they only iterate over the keys committed to the storage, but replace any overwritten values from write log.

┆Issue is synchronized with this Asana task by Unito

@tzemanovic
Copy link
Member Author

not sure if this one is actually a "good first issue", as the host env implementation employs some less-common concepts

@tzemanovic tzemanovic removed the good first issue Good for newcomers label Feb 3, 2022
@sync-by-unito sync-by-unito bot closed this as completed Feb 3, 2022
@juped juped reopened this Feb 3, 2022
@tzemanovic tzemanovic transferred this issue from anoma/anoma Jul 8, 2022
@tzemanovic
Copy link
Member Author

this is blocking #740

@juped juped closed this as completed in #913 Feb 7, 2023
@github-project-automation github-project-automation bot moved this from Todo to Tested in Devnet in Namada-Old Feb 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ledger prio:high
Projects
No open projects
Status: Tested in Devnet
Development

Successfully merging a pull request may close this issue.

2 participants