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

Remove deposit script UTxO from the local state #1688

Closed
v0d1ch opened this issue Oct 7, 2024 · 0 comments · Fixed by #1692
Closed

Remove deposit script UTxO from the local state #1688

v0d1ch opened this issue Oct 7, 2024 · 0 comments · Fixed by #1692
Assignees
@v0d1ch
Labels
follow-up Leftover work from a bigger item, ideally done in short succession.

Comments

@v0d1ch
Copy link
Contributor

v0d1ch commented Oct 7, 2024

Why

Follow up from the off-chain work on incremental commits.
We should have the needed information to spend a deposit output already kept in the chain UTxO state so there is no need to keep the local state around.

What

Remove the map of pending deposits and just keep a list of deposit TxId

How

just do it!
@v0d1ch v0d1ch added the follow-up Leftover work from a bigger item, ideally done in short succession. label Oct 7, 2024
@v0d1ch v0d1ch self-assigned this Oct 7, 2024
@v0d1ch v0d1ch changed the title Remove deposit script UTxO in from the local state Remove deposit script UTxO from the local state Oct 7, 2024
@ch1bo ch1bo linked a pull request Oct 10, 2024 that will close this issue
Don't keep deposit script utxo in local state #1692
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@v0d1ch v0d1ch

Labels
follow-up Leftover work from a bigger item, ideally done in short succession.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Don't keep deposit script utxo in local state cardano-scaling/hydra
1 participant
@v0d1ch