Add document for running JS IPFS in the browser - #2392
Conversation
|
Thanks @tapaswenipathak I'll get round to reviewing this asap! |
daviddias
left a comment
daviddias
left a comment
There was a problem hiding this comment.
@tapaswenipathak this is a very cool draft! It does pack a few things that are incoherent. I left a few comments :)
|
@daviddias: you can review now. |
Co-Authored-By: David Dias <daviddias.p@gmail.com>
Co-Authored-By: David Dias <daviddias.p@gmail.com>
lidel
left a comment
lidel
left a comment
There was a problem hiding this comment.
Thanks @tapaswenipathak! I suggested some changes to make it more aligned with how things look like on the technical side:
Co-Authored-By: Marcin Rataj <lidel@lidel.org>
Co-Authored-By: Marcin Rataj <lidel@lidel.org>
Co-Authored-By: Marcin Rataj <lidel@lidel.org>
Co-Authored-By: Marcin Rataj <lidel@lidel.org>
Co-Authored-By: Marcin Rataj <lidel@lidel.org>
|
@lidel: Very creative statements/writeups; you can review now. |
Reorganized the content a bit and added references.
lidel
left a comment
lidel
left a comment
There was a problem hiding this comment.
Thank you @tapaswenipathak!
I changed the order of the document a bit, and added references to external resources with more info to make this reusable on our docs portal as well (at some point we will move this content to https://docs.ipfs.io/reference/js/overview/, but for now let's keep it here)
LGTM, but I would like someone else to proof-read it before we merge.
|
This is a great document! It answers many questions in a concise way. Any reason to not merge this as it is, and continue to iterate from there? |
6 participants
Fixes #2231.