Create Simple Chat App Update #1161
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…on, added warning about circuit-relay v1 removal from go-ipfs & linked to a repo with most of the code needed to use the chat app /w a relay (but missing pubsub & the ability to list addrs, so no announcing) fixes #1146
johnnymatthews
approved these changes
May 26, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Updated diagram, changed links to be up-to-date, removed Docker section, added warning about circuit-relay v1 removal from go-ipfs & linked to a repo with most of the code needed to use the chat app /w a relay (but missing pubsub & the ability to list addrs, so no announcing)
fixestowards #1146This simply makes the guide correct again, however doesn't instruct the user on how to setup a circuit-relay anymore. The reason for this being is go-libp2p-relay-daemon doesn't currently have any way to:
So unless I missed something, while it's awesome having most of the hard work needed done, it's still not enough to bring life to using circuit-relay in a browser right now, not that I can figure out anyways. In the future I or someone else can likely take the Python script which is used for advertising currently, and build a custom libp2p node that bundles the pubsub announces right into it, which would completely solve the problem.