Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Render connected peers in the chat UI #21

Closed
Tracked by #35
2color opened this issue Apr 4, 2023 · 1 comment · Fixed by #147
Closed
Tracked by #35

Render connected peers in the chat UI #21

2color opened this issue Apr 4, 2023 · 1 comment · Fixed by #147

Comments

@2color
Copy link
Collaborator

2color commented Apr 4, 2023

Background

As a user, I want to know who else is in the chatroom.

We should render a list of all the other peers in the gossipsub channel.

https://github.com/feross/p2p-graph is a cool way to explore rendering the graph visually.

Details

  • Note that this chat peers may be different to the connected peers. Indeed, many of the peers you will connect to are gonna be on this topic but you may be also connected to bootstrap nodes which aren't listening on the topic and shouldn't be rendered.
@p-shahi
Copy link
Member

p-shahi commented Apr 7, 2023

Additionally:

On the gossipsub channel, we can gossip connection events i.e. “I’m a rust-peer and I’m connected to a,b,c peers over the [WebRTC/QUIC/etc] protocols” and have a visualization that shows a graph on the browser side who is connected to who over which protocols. Any browser peer would eventually learn this and be able to visualize it since this info would be gossiped frequently (every 2 or 3 secs). The graph can be built as peers join, we can give peers names.

@maschad maschad assigned jochasinga and maschad and unassigned jochasinga Apr 14, 2023
@maschad maschad removed their assignment Apr 24, 2023
2color added a commit that referenced this issue May 8, 2024
2color added a commit that referenced this issue May 8, 2024
fixes #21

Co-authored-by: Daniel N <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants