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

feat: drep expiration #240

Merged
merged 5 commits into from
Jan 17, 2025
Merged

feat: drep expiration #240

merged 5 commits into from
Jan 17, 2025

Conversation

slowbackspace
Copy link
Contributor

@slowbackspace slowbackspace commented Jan 15, 2025

prereq blockfrost/openapi#395
Added new DRep fields:

  • retired (true if drep was deregistered)
  • expired - new thing, Indicates whether the DRep has been inactive for a consecutive number of epochs (determined by a epoch parameter drep_activity)
  • last_active_epoch - Epoch of the most recent action - registration, update, deregistration or voting (replacement for deprecated active_epoch which is broken.)

Deprecated:

  • active - confusing as it only reported whether the drep is registered or not (info introduced as a new field retired)
  • active_epoch - also updated openapi description as it did not match the reality, new (correct) description: Epoch of the most recent registration

@slowbackspace
Copy link
Contributor Author

one last bug, special dreps (eg. drep_always_no_confidence) are reported as retired due to not having a registration. 🚧 👷

@slowbackspace slowbackspace marked this pull request as draft January 15, 2025 15:39
@slowbackspace slowbackspace marked this pull request as ready for review January 16, 2025 11:04
@vladimirvolek vladimirvolek self-requested a review January 17, 2025 15:41
@vladimirvolek vladimirvolek merged commit 8807ca2 into master Jan 17, 2025
1 of 2 checks passed
@vladimirvolek vladimirvolek mentioned this pull request Jan 17, 2025
@vladimirvolek vladimirvolek deleted the fix/drep-expiration branch January 17, 2025 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants