nextcloud-deck icon indicating copy to clipboard operation
nextcloud-deck copied to clipboard

Allow to delete "file" attachments

Open stefan-niedermann opened this issue 3 years ago • 3 comments

A quick test showed, that deleting the new "file" attachments via the Android app doesn't do anything on the server side. I therefore temporarily hid the delete button.

  • [ ] Reenable the delete button
  • [ ] Make sync great again

stefan-niedermann avatar Mar 11 '21 21:03 stefan-niedermann

(Documentation for later implementation, need to leave now)

Looks like the ID of the file is kind of different. I tracked the Web-UI call and saw this:

DELETE | https://myFancyURL.tld/index.php/apps/deck/cards/611/attachment/file:55

I guess the prefix for the ID has to be file:, tho I can't tell why the path is different to the official documentation.

Worth a try!

desperateCoder avatar Mar 20 '21 12:03 desperateCoder

Did some work on the 871-delete-attachments-for-1-3 branch. Though i think reverse engineering the Web UI isn't really a good idea. The URL starts with /apps/deck for example, instead of /api/deck. I differentiate on the branch so we can easily add the file: if necessary, but simply adding file: to the old API doesn't work (HTTP 405) unfortunately.

stefan-niedermann avatar Mar 20 '21 18:03 stefan-niedermann

Issue in the server app: https://github.com/nextcloud/deck/issues/2904

stefan-niedermann avatar Mar 22 '21 12:03 stefan-niedermann

Fixed shipped in Deck Android 1.24.0.

stefan-niedermann avatar Jan 22 '24 07:01 stefan-niedermann