library icon indicating copy to clipboard operation
library copied to clipboard

Trashed documents show up on site

Open ericanastas opened this issue 1 year ago • 3 comments

Context (Environment)

Node 18.14.0 Folder source

Expected Behavior

If documents are moved to the trash, and no longer show up in a folder on the google drive interface, I would expect the document to no longer show up on the site as well.

Actual Behavior

Trashed documents continue to show up on the site until they are permanently deleted.

image

image

To Reproduce

Move a document to the trash.

Possible Solution

The getOptions() in search.js includes AND trashed = false, however this filter is not included in the corresponding getOptions() function in list.js.

These options should include the trashed = false filter as well.

ericanastas avatar Mar 10 '23 00:03 ericanastas

Hi @ericanastas, thanks for the report.

As a workaround until this is fixed, you can create a top-level folder called Trash and move files you wish to hide on the site there. This was originally implemented as user permissions around deleting files can be messy within shared drives.

afischer avatar Mar 28 '23 12:03 afischer

Can you elaborate on how the permissions for deleting files can be "messy" in GSDs? Maybe there's something I'm not considering. My understanding is only drive managers can permanently delete files from the trash right? I think most users would expect files that have been moved to the trash to no longer show up on the site.

ericanastas avatar Mar 28 '23 13:03 ericanastas

I'm not sure if this is still the case, but it used to be that users that did not originally own files in shared drives could not delete them, even if they were able to edit or move them.

It's very possible Google has changed the way that permissions work for files in a shared drive in the past few years, so It's definitely worth revisiting.

afischer avatar Mar 28 '23 14:03 afischer