centrifuge / centrifuge-chain

Centrifuge Chain: the layer-1 blockchain for real-world assets, built on Substrate.
https://centrifuge.io
GNU Lesser General Public License v3.0
184 stars 81 forks source link

LPv2: Batch Message serialization #1920

Closed lemunozm closed 4 months ago

lemunozm commented 4 months ago

Description

This PR is related to the batch message support. The batch logic itself will be in a next PR.

Serialization Notes

We need a custom serialization/deserialization due to the spetial requirements of how batches needs to be serialized:

Changes

Next PR

lemunozm commented 4 months ago

As a first step for the bench message feature, this should be ready.

codecov[bot] commented 4 months ago

Codecov Report

Attention: Patch coverage is 58.06452% with 26 lines in your changes missing coverage. Please review.

Project coverage is 46.15%. Comparing base (976eebe) to head (a771a96).

Files Patch % Lines
pallets/liquidity-pools/src/message.rs 54.71% 24 Missing :warning:
pallets/liquidity-pools/src/gmpf/error.rs 33.33% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## feat/lp-v2 #1920 +/- ## ============================================== + Coverage 45.97% 46.15% +0.17% ============================================== Files 179 179 Lines 12791 12849 +58 ============================================== + Hits 5881 5930 +49 - Misses 6910 6919 +9 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

lemunozm commented 4 months ago

@mustermeiszer @wischli this is ready to review/merge. The logic itself goes in a nex PR