Add files deletion when destroying attachment, workspace or workspaceMember

[etiennejouan]

Solution

• update attachment soft delete logic by destroy (seen with Weiko & Felix)
• add two jobs for file and workspace folder deletion
• add listener to attachment and workspaceMember destroy event -> add file deletion job
• update logic in deleteWorkspace method -> add wokspace folder deletion job

closes twentyhq/core-team-issues#147

To go further

• delete old avatar when workspaceMember replaces its avatar
• same with workspace picture

[greptile-apps]

PR Summary

This PR implements a comprehensive file cleanup system for handling file deletions when attachments, workspace members, or workspaces are destroyed.

• Added FileAttachmentListener and FileWorkspaceMemberListener in /packages/twenty-server/src/engine/core-modules/file/listeners/ to handle file cleanup on entity deletion
• Implemented FileDeletionJob and FileWorkspaceFolderDeletionJob in /packages/twenty-server/src/engine/core-modules/file/jobs/ for queued file deletion operations
• Enhanced FileService with deleteFile and deleteWorkspaceFolder methods for safe file system operations
• Updated attachment handling in /packages/twenty-front/src/modules/activities/files/components/AttachmentRow.tsx to use useDestroyOneRecord instead of soft deletion
• Modified WorkspaceService to include file cleanup when deleting workspaces

Note: Consider adding error handling for file system operations and handling null/undefined avatar URLs in the workspace member listener.

_{8 file(s) reviewed, 10 comment(s)
Edit PR Review Bot Settings | Greptile}

[FelixMalfait]

Great work! You can merge and ignore the queue comment if you want, not sure if it's a must

[github-actions]

Thanks @etiennejouan for your contribution!
This marks your 26th PR on the repo. You're top 2% of all our contributors 🎉
See contributor page - Share on LinkedIn - Share on Twitter