-
Notifications
You must be signed in to change notification settings - Fork 119
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
car delete fails with 404 #1202
Comments
Hey @ylempereur I sadly could not replicate on my end. Can you give us more information on the error, like a screenshot? How long does it take from upload end to attempt to delete? |
Here is an example of how I upload my car files (token redacted):
The file appears on the Files page and eventually gets pinned and gets some providers. Later on (days, weeks), after I upload a new version, I try to delete it using the Files page (tried to delete 1, tried to delete multiples, tried 3 different browsers), and the UI freezes after the confirmation dialog. Here is what appears on the console within 1 second: Please let me know if you need further information. |
hey @vasco-santos I just had a thought: my car files use v0 CIDs internally, and your UI uses v1 CIDs, which it uses to call the delete api. Could this mismatch be the reason it fails to find the file to delete?... |
hey @vasco-santos
Thanks for the notes, it looks like it is! |
I created #1242 to fix that |
Thank you. Please let me know when the fix is live. |
Hey @ylempereur Apologies for taking so long to get back to this, a solution was not deployed to use the CIDv0 and deletes should now be functional. Can you please confirm on your end? |
I can confirm that the problem has been fixed and that delete now works as expected. Thank you for addressing this :) |
If I upload a car file using "POST /car", and then go to the "files" page and try to delete it, it fails with a 404 (the UI just fails, you have to look at the console to realize the API returned 404). This happens for any such car file, you can never delete any of them.
The text was updated successfully, but these errors were encountered: