aws-solutions / simple-file-manager-for-amazon-efs

Serverless web application to manage data in your Amazon EFS Filesystem
Apache License 2.0
79 stars 30 forks source link

remove directory feature #200

Open afom opened 1 year ago

afom commented 1 year ago

Is your feature request related to a problem? Please describe.

No ability to delete directories.

Describe the feature you'd like

Would like the ability to delete directories

simonkrol commented 1 year ago

Hi @afom, That's an interesting idea, I'll add it to the backlog to be evaluated.

How would you anticipate the delete function working for directories with files/other directories in them?

My initial thoughts would be:

  1. Directories can have files nested within other directories, so we should be careful when performing deletes.
  2. A delete of a directory with something in it should require additional explicit action (like requiring that the user type Delete into a text box) in order to complete the delete (we could make this toggle-able through a CFN parameter).
  3. Directories which are completely empty should be able to be deleted without an explicit action.

Let me know what you think, Simon