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

Remove ClientProvider in favour of PolkadotClient #1052

Merged
merged 1 commit into from
Apr 29, 2020

Conversation

seunlanlege
Copy link
Contributor

companion pr for paritytech/substrate#5823

@parity-cla-bot
Copy link

It looks like @seunlanlege signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

@seunlanlege seunlanlege requested a review from bkchr April 29, 2020 11:42
Copy link
Contributor

@gnunicorn gnunicorn left a comment

Choose a reason for hiding this comment

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

lgtm

Runtime: ConstructRuntimeApi<Block, Self>
{}

impl<Block, Backend, Runtime, Client> PolkadotClient<Block, Backend, Runtime> for Client
Copy link
Contributor

Choose a reason for hiding this comment

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

is this even needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah unfortunately it is, Substrait Client doesn't implement PolkadotClient

@gnunicorn gnunicorn added the A0-please_review Pull request needs code review. label Apr 29, 2020
@seunlanlege seunlanlege merged commit 1fce54e into master Apr 29, 2020
@seunlanlege seunlanlege deleted the seun-remove-client-provider branch April 29, 2020 12:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants