feat: select peer to dial from drop down #282
Conversation
adklempner
force-pushed
the
adklempner/248-light-js-debugging-tool
branch
2 times, most recently
from
6a11881
to
c8a849e
Compare
adklempner
force-pushed
the
adklempner/248-light-js-debugging-tool
branch
from
c8a849e
to
a70c28d
Compare
examples/light-js/index.html
Outdated
| // Queries all peers from store and updates list of connected peers | ||
| const updatePeersList = async () => { | ||
| // Generate <p> element with connection string from each peer | ||
| const peers = await node.store.peers(); |
There was a problem hiding this comment.
node.store refers to Store protocol and though it seems to be possible to get peers from there we should try using connectionManager or libp2p directly.
There was a problem hiding this comment.
Updated to use node.libp2p.peerStore
examples/light-js/index.html
Outdated
| <label for="remote-multiaddr">Remote peer's multiaddr</label> | ||
| </div> | ||
| <div> | ||
| <input id="remote-multiaddr" type="text" value="" style="width: 100%" /> |
There was a problem hiding this comment.
nit: let's add some styling - this field is soooo long :)
There was a problem hiding this comment.
Added a right margin to parent div and made the maximum width for the input 900px; this is about as wide it needs to be to display the entire multiaddr
adklempner
force-pushed
the
adklempner/248-light-js-debugging-tool
branch
from
a70c28d
to
2391874
Compare
|
Is that related to a specific issue? |
|
it checks off 3 of the boxes listed as requirements for this issue: #248 (comment) |
Ah great. I recommend to mention the issue in the description of PRs. |
No description provided.