Skip to content
This repository has been archived by the owner on Oct 10, 2023. It is now read-only.

change OnChainTreasuryID to T::TreasuryId for bank module #91

Closed
wants to merge 17 commits into from

Conversation

4meta5
Copy link
Collaborator

@4meta5 4meta5 commented Jun 15, 2020

despite the Codec bound on the associated type, this error was returned.

error[E0277]: the trait bound `<T as Trait>::TreasuryId: _::_parity_scale_codec::EncodeLike` is not satisfied
   --> modules/bank/src/lib.rs:125:1
    |
125 | / decl_storage! {
126 | |     trait Store for Module<T: Trait> as Bank {
127 | |         /// Counter for generating uniqe bank accounts
128 | |         BankIDNonce get(fn bank_id_nonce): T::TreasuryId;
...   |
155 | |     }
156 | | }
    | |_^ the trait `_::_parity_scale_codec::EncodeLike` is not implemented for `<T as Trait>::TreasuryId`
    |
    = note: this error originates in a macro (in Nightly builds, run with -Z macro-backtrace for more info)

error: aborting due to previous error

For more information about this error, try `rustc --explain E0277`.
error: could not compile `sunshine-bank`.

To learn more, run the command again with --verbose.

4meta5 added 17 commits June 11, 2020 14:31
…is unusable, will add tests the same way as substrate identity example in future commits
…ont actually watch event emission and it is missing that
@4meta5
Copy link
Collaborator Author

4meta5 commented Jun 16, 2020

stale now because I merged #89 which adds a lot on top of this but the last commit may be useful if we ever decide to make this change in the future

@4meta5 4meta5 closed this Jun 16, 2020
@4meta5 4meta5 deleted the failed-attempt-to-make-treasury-id-generic branch June 16, 2020 22:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant