-
Notifications
You must be signed in to change notification settings - Fork 324
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
Context action to "Add to MFS" #635
Comments
No, don't pin! Pinning is a great maze of despair. Once pinned, you're guaranteed to never find that thing ever again, and also guaranteed to not be able to delete it if you want. But adding to MFS + actually download the object is ideal.
I like this name. |
Definitely a great idea!
If presence in MFS is protects data from GC, that seems is quite enough. (To prevent the terrible things that @fiatjaf is talking about, pure pinning would require some kind of external pin log...)
What do you think about
Let's consider this point in a more generic way! In my opinion (not sure whether this opinion is shared by IPFS team or not), all IPFS-enabled applications should has a knowledge about which data they added to IPFS. Just pass data to IPFS daemon is not seems to be enough. It could be implemented as unique internal storage on the side of every of those applications — but it also could be a part of shared IPFS API, which may be easily adopted by certain apps. The second way looks more acceptable, isn't it? Thereby, it doesn't much sense to keep menu item that using ...But some kind of upload log that also could be stored in MFS would be pretty useful. This log should contain at least the original file name, date of upload and the original address of added resource. |
Cool idea!
We've actually are been thinking about switching Upload and all "Add to IPFS" context actions to MFS.
Would appreciate some feedback:
(I'd say to not pin: MSF is enough, and if we remove file from MFS it can be GC'd)
/
) as it may get noisy quite fast. Perhaps/added-via-ipfs-companion/
? (looking for better name)The text was updated successfully, but these errors were encountered: