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

Use ADA Pots Leger Event #1258

Closed
kderme opened this issue Sep 13, 2022 · 1 comment
Closed

Use ADA Pots Leger Event #1258

kderme opened this issue Sep 13, 2022 · 1 comment
Labels
bug Something isn't working

Comments

@kderme
Copy link
Contributor

kderme commented Sep 13, 2022

Ledger has recently added a new ledger event for AdaPots IntersectMBO/cardano-ledger#2797. It gives the AdaPots after the Epoch Boundary, but before the first block is applied. We should use it on the db-sync side, to avoid issues describe in #942

@kderme kderme added the bug Something isn't working label Sep 13, 2022
@kderme kderme added the good first issue Good for newcomers label Sep 19, 2022
@kderme kderme removed the good first issue Good for newcomers label Oct 4, 2022
@kderme kderme moved this to In Progress in DBSync Board Dec 14, 2022
kderme added a commit that referenced this issue Feb 27, 2023
Instead of using the ledger. This changes semantically the values of
the DB, so it's a c level breaking change. A migration is impossible to
cover these changes. Fixes #1258.
@kderme kderme moved this from In Progress to Blocked in DBSync Board Mar 27, 2023
kderme added a commit that referenced this issue May 11, 2023
Instead of using the ledger. This changes semantically the values of
the DB, so it's a c level breaking change. A migration is impossible to
cover these changes. Fixes #1258.
kderme added a commit that referenced this issue May 11, 2023
kderme added a commit that referenced this issue May 11, 2023
Instead of using the ledger. This changes semantically the values of
the DB, so it's a c level breaking change. A migration is impossible to
cover these changes. Fixes #1258.
@kderme kderme moved this from Blocked to In Progress in DBSync Board May 16, 2023
kderme added a commit that referenced this issue May 16, 2023
Instead of using the ledger. This changes semantically the values of
the DB, so it's a c level breaking change. A migration is impossible to
cover these changes. Fixes #1258.
kderme added a commit that referenced this issue May 16, 2023
@kderme kderme mentioned this issue May 16, 2023
9 tasks
kderme added a commit that referenced this issue May 16, 2023
Instead of using the ledger. This changes semantically the values of
the DB, so it's a c level breaking change. A migration is impossible to
cover these changes. Fixes #1258.
kderme added a commit that referenced this issue May 16, 2023
kderme added a commit that referenced this issue May 17, 2023
Instead of using the ledger. This changes semantically the values of
the DB, so it's a c level breaking change. A migration is impossible to
cover these changes. Fixes #1258.
kderme added a commit that referenced this issue May 17, 2023
kderme added a commit that referenced this issue May 17, 2023
Instead of using the ledger. This changes semantically the values of
the DB, so it's a c level breaking change. A migration is impossible to
cover these changes. Fixes #1258.
kderme added a commit that referenced this issue May 17, 2023
kderme added a commit that referenced this issue May 17, 2023
Instead of using the ledger. This changes semantically the values of
the DB, so it's a c level breaking change. A migration is impossible to
cover these changes. Fixes #1258.
kderme added a commit that referenced this issue May 17, 2023
kderme added a commit that referenced this issue May 17, 2023
Instead of using the ledger. This changes semantically the values of
the DB, so it's a c level breaking change. A migration is impossible to
cover these changes. Fixes #1258.
kderme added a commit that referenced this issue May 17, 2023
@kderme
Copy link
Contributor Author

kderme commented May 17, 2023

Fixed at 1ad4521

@kderme kderme closed this as completed May 17, 2023
@github-project-automation github-project-automation bot moved this from In Progress to Done in DBSync Board May 17, 2023
kderme added a commit that referenced this issue May 17, 2023
Instead of using the ledger. This changes semantically the values of
the DB, so it's a c level breaking change. A migration is impossible to
cover these changes. Fixes #1258.
kderme added a commit that referenced this issue May 17, 2023
Cmdv pushed a commit that referenced this issue Jun 6, 2023
Instead of using the ledger. This changes semantically the values of
the DB, so it's a c level breaking change. A migration is impossible to
cover these changes. Fixes #1258.
Cmdv pushed a commit that referenced this issue Jun 6, 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
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant