Skip to content

Conversation

@TheLedgerOfJoudi
Copy link
Contributor

Added the TradingPairSourceManager trait to pallet configs in order to avoid direct calls from the trading-pair pallet.

  1. All components get enabled sources info from TradingPairSourceManager : Yes.
  2. All tests passed: Yes.

[package]
name = "framenode"
version = "3.0.0"
version = "3.1.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

You don't need to change the version of any pallet in this scope

Comment on lines +259 to +262
spec_version: 66,
impl_version: 1,
apis: RUNTIME_API_VERSIONS,
transaction_version: 64,
transaction_version: 66,
Copy link
Contributor

Choose a reason for hiding this comment

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

Please revert it. We have a special process where we update these values

@wer1st wer1st changed the title 441 trading pair source trait Trading pair source trait Oct 27, 2023
@TheLedgerOfJoudi
Copy link
Contributor Author

I moved this draft to new PR #808, kindly check. Pumping versions appears here because I did a wrong branch-out before opening the PR.

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