Skip to content
This repository was archived by the owner on Mar 10, 2020. It is now read-only.

Conversation

vasco-santos
Copy link
Contributor

@vasco-santos vasco-santos commented Jul 23, 2019

On libp2p/js-libp2p#365, we decided to accept strings as the data being published. This usually was a barrier for entry users, so we just convert from string to buffer, if we receive a string.

Needs:

@vasco-santos vasco-santos marked this pull request as ready for review August 20, 2019 08:08
@vasco-santos vasco-santos requested a review from alanshaw August 20, 2019 08:08
Copy link
Contributor

@alanshaw alanshaw left a comment

Choose a reason for hiding this comment

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

Please would you also update the docs? 🙏

@vasco-santos vasco-santos force-pushed the chore/integrate-libp2p-new-pubsub-routing branch 2 times, most recently from 6ad700a to c92a241 Compare August 28, 2019 08:11
@vasco-santos vasco-santos force-pushed the chore/integrate-libp2p-new-pubsub-routing branch from c92a241 to 7375d8d Compare August 28, 2019 08:21
@alanshaw alanshaw merged commit acafbd1 into master Aug 28, 2019
@alanshaw alanshaw deleted the chore/integrate-libp2p-new-pubsub-routing branch August 28, 2019 08:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants