-
Notifications
You must be signed in to change notification settings - Fork 24
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
Adds handling for content in beacon network
#428
Conversation
f8ec399
to
f87f8f6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall, LGTM!
I think it's worth Typescript engineering something around the content types as we start to add more networks, but that's more about style than function.
:+1 to merge this and then work on setting up new tests
Agreed. Probably some sort of nested enum if that exists but haven't gotten that far yet. |
Adds storing and retrieving of
Beacon
network content from local storage as well as a partial implementation of the SENDFINDCONTENT flow. This currently only handles the case where the content is delivered via uTP stream.