-
Notifications
You must be signed in to change notification settings - Fork 491
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
feat: IPFS Desktop settings #883
Conversation
The injected code can be found here. |
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.
Looks good. Minor tweaks needed. Please add a storybook example.
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.
LGTM after fixing @olizilla comments!
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.
Try not to rush. Run your changes before pusing them.
License: MIT Signed-off-by: Henrique Dias <hacdias@gmail.com>
License: MIT Signed-off-by: Henrique Dias <hacdias@gmail.com>
License: MIT Signed-off-by: Henrique Dias <hacdias@gmail.com>
License: MIT Signed-off-by: Henrique Dias <hacdias@gmail.com>
1671ff9
to
25c6966
Compare
License: MIT Signed-off-by: Henrique Dias <hacdias@gmail.com>
@olizilla working and with a storybook 😄 |
License: MIT Signed-off-by: Henrique Dias <hacdias@gmail.com>
License: MIT Signed-off-by: Henrique Dias <hacdias@gmail.com>
License: MIT Signed-off-by: Henrique Dias <hacdias@gmail.com>
License: MIT Signed-off-by: Henrique Dias <hacdias@gmail.com>
License: MIT Signed-off-by: Henrique Dias <hacdias@gmail.com>
@olizilla let's merge this! Somehow it started passing 😄 I'm happy for it 😃 It would be rad if you released a new version so we can use it on IPFS Desktop right away! |
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.
Tried this out from desktop, and it works well. Just needs more polish.
@hacdias please revert the change to the package-lock.json. We need to upgrade ipld and ipld-explorer-components together or we hit breaking changes that causes errors when trying to view some ipld types. |
License: MIT Signed-off-by: Henrique Dias <hacdias@gmail.com>
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.
LGTM
More info see: ipfs/ipfs-desktop#674.
IPFS Desktop injects a bundle on
window.ipfsDesktop
. If it's present, we show IPFS Desktop specific settings.(Screenshot taken with IPFS Desktop)
License: MIT
Signed-off-by: Henrique Dias hacdias@gmail.com