Skip to content

feat: write plutus data v1 for permissioned candidates and registrations #923

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

Merged
merged 3 commits into from
Jul 28, 2025

Conversation

LGLO
Copy link
Contributor

@LGLO LGLO commented Jul 18, 2025

Description

  • Change of v1 format for permissioned candidates - it's not too late, because nothing was writing it yet
  • Implemented logic for writing: if keys are AURA and Grandpa only, then use v0, but if keys are different, then use v1

Checklist

  • Commit sequence broadly makes sense and commits have useful messages.
  • The size limit of 400 LOC isn't needlessly exceeded
  • The PR refers to a JIRA ticket (if one exists)
  • New tests are added if needed and existing tests are updated.
  • New code is documented and existing documentation is updated.
  • Relevant logging and metrics added
  • Any changes are noted in the changelog.md for affected crate
  • Self-reviewed the diff

@LGLO LGLO force-pushed the generic-keys-11-write-plutus-data branch from f20a3b6 to c88a40d Compare July 18, 2025 14:47
@LGLO LGLO requested review from kpinter-iohk and AmbientTea July 25, 2025 04:38
@LGLO LGLO force-pushed the generic-keys-11-write-plutus-data branch from 099fd55 to d2ef3f2 Compare July 25, 2025 09:43
@LGLO LGLO force-pushed the generic-keys-11-write-plutus-data branch from d2ef3f2 to 8cb6985 Compare July 28, 2025 12:11
@LGLO LGLO enabled auto-merge (squash) July 28, 2025 12:48
@LGLO LGLO merged commit 4e98a9b into master Jul 28, 2025
29 checks passed
@LGLO LGLO deleted the generic-keys-11-write-plutus-data branch July 28, 2025 12:54
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.

4 participants