-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Add ability to publish a webpage to IPFS #14770
Comments
@lidel do you have an idea on how this could be implemented? |
Depends on if you want to publish an immutable snapshot ( Former is easy, you The latter is the same, but additionally you need to have a mutable pointer to an immutable snapshot at Right now we have two ways of creating such pointers:
|
Created PR without p2p-key management because we need to clarify how it should look for users from the product side |
I started testing & verifying this using the testplan at brave/brave-core#8561 (where 3/5 cases are fine), but ran into #15634 - I'll wait for that to be resolved before coming back to fully verify this. |
@stephendonner is this issue blocked by #15634? if so, we should have this as |
Not totally blocked by that behavior (which I've since been told is expected via Chromium's, which we inherit), thanks for asking for clarification; the real problem, I think, is #15935, which @spylogsster is looking into. I'll mark it |
Verified
Check various cases when you have ongoing downloads and you do the import for a web page, it should not appear on the shelf with othersSteps:
Confirmed I only saw the original CNN page being saved to Downloads, not when IPFS sharing Check that downloaded pages are not visible in the download historySteps:
Confirmed that only the Wikipedia page I saved locally to share later via IPFS showed up in the Downloads window Check that the tab does not crash if downloading is in progressConfirmed no crashes while testing Check imported files have same content as originalsSteps:
Check imported pages open correctly by shareable linkSteps:
![]() @spylogsster @bbondy @lidel do you think this needs to be tested on all platforms, or is this abstracted/modular enough to be expected to work on macOS, Windows, and Linux? Thanks! Verification passed on
Verification passed on
Made sure folders can be imported Verification PASSED on
Check various cases when you have ongoing downloads and you do the import for a web page, it should not appear on the shelf with others
Confirmed that only the original CNN page was being displayed via
Check that downloaded pages are not visible in the download history
Confirmed that only the Wikipedia page I saved locally to share later via IPFS showed up in the Downloads window
Check imported files have same content as originals
Check imported pages open correctly by shareable linkSteps:
|
@stephendonner I am not familiar with Chromium internals to tell if this is abstracted away, but there is a potential diff around each OS having code paths for opening the file/dir picker GUI. Everything else should be the same. |
This is a placeholder to add the ability to recursively publish a website to IPFS.
Perhaps this will pin content of the current page, subresources, and some depth of links automatically.
The text was updated successfully, but these errors were encountered: