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

add parachain backing #1192

Closed
wants to merge 9 commits into from
Closed

add parachain backing #1192

wants to merge 9 commits into from

Conversation

xiaoch05
Copy link
Contributor

No description provided.

Copy link
Contributor

@hackfisher hackfisher left a comment

Choose a reason for hiding this comment

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

Any progress or ETA on this?

@@ -165,7 +165,7 @@ pub mod pallet {
origin: OriginFor<T>,
token_address: H160,
amount: U256,
recipient: H160,
recipient: Vec<u8>,
Copy link
Contributor

Choose a reason for hiding this comment

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

Are you try to generalize this?

If not that easy, I would like to keep it simple since these pallets will be reimplemented using message SDK any way, It does not deserve more time on this.

Just straight forward and keep simple, we do not have to reuse existing pallet.

cc @xiaoch05 @wuminzhe

@xiaoch05
Copy link
Contributor Author

Any progress or ETA on this?

The test has been finished.
Will ready for review after some code clean.

@xiaoch05
Copy link
Contributor Author

use #1243

@xiaoch05 xiaoch05 closed this May 31, 2022
@xiaoch05 xiaoch05 deleted the xiaoch05-parachain-backing branch August 11, 2022 01:41
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.

2 participants