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

1.2.1 #302

Merged
merged 12 commits into from
Sep 17, 2024
Merged

1.2.1 #302

merged 12 commits into from
Sep 17, 2024

Conversation

Scitz0
Copy link
Contributor

@Scitz0 Scitz0 commented Sep 11, 2024

Description

Various patches for 1.2.0

  • Fix for when both key and script was registered with same hash
  • Fix for drep_script format according to CIP-105, implementation was in conflict with pre-defined roles
  • Pool stat fix, rollback to previous code until DBSync gets a fix
  • Replace usage of view column with own bech32 utility functions
    • Utility function for stake_address encoding/decoding
    • stake_address.view references
    • Cache data that rely on bech32 format for stake account
    • Any cache references that rely on stake account in bech32 format
    • pool_hash.view references
    • Cache data including bech32 format for pool IDs
    • Any cache references that rely on pool ID in bech32 format

Cancelled:

  • DRep view references - cancelled due to presence of pre-created roles in dbsync
  • multi_asset > fingerprint references - requires addition of blake2b extension, TBC - if/when dbsync retires bech32
  • Truncate (delete from) multi_asset.fingerprint, stake_address.view column and corresponding indexes to save on disk space and background maintenance on Postgres DB => In absence of tx_out.address, the primary benefactor of this would be multi_asset.fingerprint, but that's currently not being handled in this release - Hence, skipped this item accordingly

Scitz0 and others added 5 commits September 10, 2024 23:09
- Fix for when both key and script has was registered with same hash
- Fix for drep_script format according to CIP-105, implementation was in conflict with pre-defined roles
@rdlrt rdlrt merged commit 7c6a243 into main Sep 17, 2024
2 checks passed
@rdlrt rdlrt deleted the 1.2.1 branch September 17, 2024 14:43
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.

3 participants