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

Use libp2p rather than devp2p for network subsystem #187

Closed
gavofyork opened this issue May 30, 2018 · 9 comments
Closed

Use libp2p rather than devp2p for network subsystem #187

gavofyork opened this issue May 30, 2018 · 9 comments
Assignees
@tomaka
Labels
U2-some_time_soon Issue is worth doing soon.
Milestone
PoC-2 (Co-finalis...

Comments

@gavofyork
Copy link
Member

No description provided.

@gavofyork gavofyork added M4-core U2-some_time_soon Issue is worth doing soon. Z2-medium Can be fixed by a coder with good Rust knowledge but little knowledge of the codebase. labels May 30, 2018
@gavofyork gavofyork added this to the PoC-2 (Co-finalisation) milestone May 30, 2018
@tomaka
Copy link
Contributor

tomaka commented May 30, 2018

I have a branch with a working prototype here: https://github.com/tomaka/polkadot/tree/libp2p

However this prototype uses my WIP code for integrating libp2p in Parity. I have various issues with that because it collides with devp2p.

If PoC-2 is to come soon, I would just move the code to polkadot and concentrate on it instead.

@gavofyork
Copy link
Member Author

@tomaka please go ahead and move the code :)

@gitcoinbot
Copy link

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

This issue now has a funding of 250.0 DAI (250.0 USD @ $1.0/DAI) attached to it.

@vs77bb
Copy link

vs77bb commented Jun 20, 2018

@tomaka It looks like you may have made some initial progress here, interested in claiming the Gitcoin bounty here?

@tomaka
Copy link
Contributor

tomaka commented Jun 21, 2018

@vs77bb I'm employed by Parity, so bounty or not I'm doing this anyway 🙃

This was referenced Jun 28, 2018
Closed
Merged
@timxor
Copy link

timxor commented Jul 6, 2018

Is this being worked on or does it still need to be done? What were the explicit requirements?

@rphmeier
Copy link
Contributor

rphmeier commented Jul 6, 2018

@tcsiwula it's assigned to @tomaka : see #268

@gitcoinbot
Copy link

Issue Status: 1. Open 2. Cancelled

The funding of 250.0 DAI (250.0 USD @ $1.0/DAI) attached to this issue has been cancelled by the bounty submitter

@vs77bb
Copy link

vs77bb commented Jul 10, 2018

Thanks for clarification @tomaka 😄- pulled the bounty off here so the funds can be used elsewhere!

@gavofyork gavofyork removed the Z2-medium Can be fixed by a coder with good Rust knowledge but little knowledge of the codebase. label Jul 13, 2018
lamafab pushed a commit to lamafab/substrate that referenced this issue Jun 16, 2020
@gabreal
Ci kubernetes chart update (paritytech#187)
6696c8e 
* update helm chart from substrate

* ci: change container image to kubetools
JoshOrndorff added a commit to moonbeam-foundation/substrate that referenced this issue Apr 21, 2021
@JoshOrndorff
Update pallet-etheruem-chain-id to FRAME V2 (paritytech#187)
ae75d2b
liuchengxu pushed a commit to chainx-org/substrate that referenced this issue Aug 23, 2021
@gguoss
Merge pull request paritytech#187 from chainpool/develop
47104f7 
Develop
liuchengxu added a commit to chainx-org/substrate that referenced this issue Aug 23, 2021
@atenjin @liuchengxu
Refactor/assets (paritytech#187)
3ba6ca3 
* deny native asset in asset module

* refactor:split assets as assets-metadata and assets

* fix all compile error

* improve asset restrictions

* fix test for dex and asset_mining

* rename precision to decimals

* rename other precision

* Rename crate name to assets-registrar

* Rename to register/deregister

There is no need to use `register_asset` as it's clear we in the
contenxt of asset.

* Split out registrar/verifier

* Commit verifier.rs

* No need to use a marco as it's used only once now

* Rename OnAssetRegisterOrRevoke to RegistrarHandler

* Refactor asset_ids()

* Extract is_valid_asset() and asset_exists()

* Rename to get_asset_info()

* Return an iterator instead of BTreeMap

* Add some docs

* Nits

* rename AssetType Free to Usable

* some fix

* Simplify the tuple impl for RegistrarHandler

* for clippy

* Fix event generic type

Co-authored-by: Liu-Cheng Xu <[email protected]>
liuchengxu pushed a commit to autonomys/substrate that referenced this issue Jun 3, 2022
@ozgunozerk
Merge pull request paritytech#187 from subspace/Readme-Update
74c6f0e 
Readme update
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@tomaka tomaka

Labels
U2-some_time_soon Issue is worth doing soon.
Projects
None yet
Milestone
PoC-2 (Co-finalisation)
Development

No branches or pull requests
6 participants
@gavofyork @tomaka @timxor @rphmeier @vs77bb @gitcoinbot