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

fix: ability to create deleted file/directory again #216

Merged
merged 5 commits into from
Mar 13, 2023

Conversation

IgorShadurin
Copy link
Collaborator

This fix fixes deleting and re-uploading not only files, but also directories.

Copied FairOS mechanism for removing content. A feed containing the file's metadata is replaced with the magic word __Fair__ after deletion in order to be able to detect that the file has already been uploaded.

Close #192

@IgorShadurin IgorShadurin marked this pull request as ready for review February 14, 2023 20:36
@IgorShadurin IgorShadurin requested a review from nugaon as a code owner February 14, 2023 20:36
Copy link
Collaborator

@nugaon nugaon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You put the feed lookup into an ambiguous state like this.

src/feed/api.ts Outdated Show resolved Hide resolved
src/account/mnemonic.ts Outdated Show resolved Hide resolved
src/content-items/handler.ts Show resolved Hide resolved
src/feed/api.ts Outdated Show resolved Hide resolved
src/content-items/handler.ts Outdated Show resolved Hide resolved
@nugaon nugaon self-requested a review February 22, 2023 20:20
Copy link
Collaborator

@nugaon nugaon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if you use the "magic word" to flag the deleted element then document that in the personal storage doc.

src/content-items/handler.ts Outdated Show resolved Hide resolved
src/feed/api.ts Outdated Show resolved Hide resolved
@IgorShadurin IgorShadurin force-pushed the 192-new-upload-deleted-file-again branch from ca13430 to 0ca8e75 Compare March 6, 2023 15:15
@IgorShadurin
Copy link
Collaborator Author

if you use the "magic word" to flag the deleted element then document that in the personal storage doc.

Which doc? Readme or FIP?

@IgorShadurin IgorShadurin requested a review from nugaon March 6, 2023 15:39
Copy link
Collaborator

@nugaon nugaon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

please create new PR to update https://github.com/fairDataSociety/FIPs/blob/master/text/0061-personal-storage.md regarding the deletion

src/feed/api.ts Outdated Show resolved Hide resolved
@IgorShadurin
Copy link
Collaborator Author

Added info about deletion: fairDataSociety/FIPs#70

@IgorShadurin IgorShadurin merged commit 988f9e8 into master Mar 13, 2023
@IgorShadurin IgorShadurin deleted the 192-new-upload-deleted-file-again branch March 13, 2023 08:40
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

Successfully merging this pull request may close these issues.

fdp.file.uploadData does not allow overwrites
2 participants