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

Readme Typo? #5

Open
ROBERT-MCDOWELL opened this issue Aug 10, 2021 · 4 comments
Open

Readme Typo? #5

ROBERT-MCDOWELL opened this issue Aug 10, 2021 · 4 comments

Comments

@ROBERT-MCDOWELL
Copy link

You can use the upload event to achieve this:

await nebulus.connect()
nebulus.on("push", (cid) => {
// do something here
})

Because the upload event fires when a CID is successfully found on an IPFS gateway (ipfs.io), you can be sure that it will be pinned for at least a while.

But in your code the event is "push", not "upload", did I miss something?

@skogard
Copy link
Collaborator

skogard commented Aug 10, 2021

Thanks for pointing it out. I originally had named them upload and download but transitioned to push and pull concepts. They do the same thing, but I find that the push and pull based git analogy is much more intuitive and straightforward.

But forgot to update some part of the doc and just fixed it. Here's the diff: f7f425f

@ROBERT-MCDOWELL
Copy link
Author

ROBERT-MCDOWELL commented Aug 10, 2021

ok great, sure push pull is more intuitive. however it would be maybe useful for some to describe that "push" includes "other terms commonly used in web and html world like upload etc.." same for "pull" etc...
btw, your project is very promising, as it opens the door to an async full mesh web network and add another layer of encryption for data packets which can be detached from ipfs itself. I guess it will be easier to manage real time packets for live streaming and buffer for example, but many many other cases too. I hope a lot of developers will come to support nebulus.

@skogard
Copy link
Collaborator

skogard commented Aug 11, 2021

Thanks, yeah I agree it would be great if it included more explanation. Feel free to send a PR, the documentation is here https://github.com/skogard/nebulus/blob/main/docs/README.md

Otherwise I'll try to add some more details next time I update the repo.

@ROBERT-MCDOWELL
Copy link
Author

I think it's better you do it, as I'm pretty new here and you know much better your code than me :)

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

No branches or pull requests

2 participants