Skip to content

feat(drive): log number of refunded epochs#729

Merged
shuplenkov merged 7 commits intov0.24-devfrom
refunded_ecpochs_log
Jan 17, 2023
Merged

feat(drive): log number of refunded epochs#729
shuplenkov merged 7 commits intov0.24-devfrom
refunded_ecpochs_log

Conversation

@shuplenkov
Copy link
Copy Markdown

Issue being fixed or feature implemented

Log number of refunded epochs in drive

What was done?

Additional refundedEpochsCount field was added to logs

How Has This Been Tested?

With tests

Breaking Changes

No

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation

For repository code-owners and collaborators only

  • I have assigned this pull request to a milestone

@shuplenkov shuplenkov added this to the v0.24.0 milestone Jan 13, 2023
Comment thread packages/rs-drive-abci/src/execution/fee_pools/fee_distribution.rs Outdated
Konstantin Shuplenkov added 2 commits January 17, 2023 17:03
Comment thread packages/js-drive/lib/abci/handlers/proposal/endBlockFactory.js Outdated
Comment thread packages/rs-drive-abci/src/abci/messages.rs Outdated
shumkov
shumkov previously approved these changes Jan 17, 2023
Copy link
Copy Markdown
Collaborator

@shumkov shumkov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Copy Markdown
Collaborator

@shumkov shumkov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@shuplenkov shuplenkov merged commit f44a0db into v0.24-dev Jan 17, 2023
@shuplenkov shuplenkov deleted the refunded_ecpochs_log branch January 17, 2023 15:37
shumkov added a commit that referenced this pull request May 5, 2026
…e PR

dashcore PR #729 (dashpay/rust-dashcore#729) is
the companion to #708 — same `ContentDeserializer` HR-quirk root cause,
but for the separate `hashes::serde_macros::SerdeHash` macro family
that generates `Txid` / `BlockHash` / `ProTxHash` / `PubkeyHash` /
`QuorumHash` etc. (vs. #708 which fixed `OutPoint` via
`serde_struct_human_string_impl!`).

Update the two `#[ignore]` notes on `Validator::value_round_trip` and
`ValidatorSet::value_round_trip` to reference #729 instead of the vague
"follow-up PR" phrasing. When #729 lands and we bump dashcore, drop the
`#[ignore]`s.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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