papermark icon indicating copy to clipboard operation
papermark copied to clipboard

[🕹️] Add the ability to delete folders

Open mfts opened this issue 1 year ago • 23 comments

What side quest or challenge are you solving?

Currently, we can only delete folders that are empty.

We want to be able to delete folders that contain documents and perhaps other folders with documents. There should be a chain delete reaction if the user decides to delete the parent folder.

Considerations:

  • a folder can be empty
  • a folder can contain documents
  • a folder can contain nested folders with or without documents
  • there should be a strong UI prompt showing how many files will also be deleted, if files are in a dataroom and that it deletes all data and analytics.
  • UI should update and give proper feedback after the folder has been deleted

Similar to the delete team UI modal

CleanShot 2024-10-14 at 15 26 14@2x

We are looking for considerate, stable implementation.

Points

750 Points

Provide proof that you've completed the task

Open a PR with the working code and a detailed explanation of what you did, the use cases you tested and if possible, a video showing the solution.

mfts avatar Oct 14 '24 06:10 mfts

/assign

vamus092 avatar Oct 14 '24 06:10 vamus092

Assigned to @vamus092! Please open a draft PR linking this issue within 48h ⚠️ If we can't detect a PR from you linking this issue in 48h, you'll be unassigned automatically 🕹️ Excited to have you ship this 🚀

oss-gg[bot] avatar Oct 14 '24 06:10 oss-gg[bot]

/assign

roshnithakor07 avatar Oct 14 '24 08:10 roshnithakor07

This issue is already assigned to another person. Please find more issues here.

oss-gg[bot] avatar Oct 14 '24 08:10 oss-gg[bot]

/assign

sweets-11 avatar Oct 14 '24 20:10 sweets-11

This issue is already assigned to another person. Please find more issues here.

oss-gg[bot] avatar Oct 14 '24 20:10 oss-gg[bot]

/assign

prajyot004 avatar Oct 15 '24 06:10 prajyot004

This issue is already assigned to another person. Please find more issues here.

oss-gg[bot] avatar Oct 15 '24 06:10 oss-gg[bot]

/assign

nil-frontend avatar Oct 15 '24 16:10 nil-frontend

This issue is already assigned to another person. Please find more issues here.

oss-gg[bot] avatar Oct 15 '24 16:10 oss-gg[bot]

/assign

Pk9697 avatar Oct 16 '24 09:10 Pk9697

Assigned to @Pk9697! Please open a draft PR linking this issue within 48h ⚠️ If we can't detect a PR from you linking this issue in 48h, you'll be unassigned automatically 🕹️ Excited to have you ship this 🚀

oss-gg[bot] avatar Oct 16 '24 09:10 oss-gg[bot]

/assign

manisprasad avatar Oct 17 '24 16:10 manisprasad

This issue is already assigned to another person. Please find more issues here.

oss-gg[bot] avatar Oct 17 '24 16:10 oss-gg[bot]

/assign

harshraj1512 avatar Oct 17 '24 19:10 harshraj1512

This issue is already assigned to another person. Please find more issues here.

oss-gg[bot] avatar Oct 17 '24 19:10 oss-gg[bot]

Hey @mfts, Can I work on this? I've another issue under review (finished) - could you assign this to me so I can finish this one as well? I've done this kinda work before :)

Khaan25 avatar Oct 18 '24 05:10 Khaan25

/assign

AndrewHamal avatar Oct 20 '24 10:10 AndrewHamal

Assigned to @AndrewHamal! Please open a draft PR linking this issue within 48h ⚠️ If we can't detect a PR from you linking this issue in 48h, you'll be unassigned automatically 🕹️ Excited to have you ship this 🚀

oss-gg[bot] avatar Oct 20 '24 10:10 oss-gg[bot]

@mfts?

Khaan25 avatar Oct 20 '24 10:10 Khaan25

@mfts does this look fine while deleting the folder

Screenshot 2024-10-20 at 8 41 27 PM

AndrewHamal avatar Oct 20 '24 14:10 AndrewHamal

/assign

Nithin-532 avatar Oct 22 '24 10:10 Nithin-532

This issue is already assigned to another person. Please find more issues here.

oss-gg[bot] avatar Oct 22 '24 10:10 oss-gg[bot]