Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: Improve files deletion #143

Merged
merged 2 commits into from
Nov 14, 2023
Merged

Fix: Improve files deletion #143

merged 2 commits into from
Nov 14, 2023

Conversation

mguihal
Copy link
Collaborator

@mguihal mguihal commented Nov 8, 2023

Hello,

Faisant suite à ma PR Semapps assemblee-virtuelle/semapps#1197 qui gère les cas de suppression de fichier quand on en uploade un nouveau dans le formulaire, cette PR gère cette fois-ci la correcte suppression des fichiers quand on supprime une ressource.

  • Actuellement, un fichier n'est attaché qu'à une ressource, donc il n'y a pas de risque à le supprimer quand la ressource est détruite
  • On ne peut pas faire cette logique dans Semapps, parce que le data-provider ne connait pas quel champ de quelle ressource contient un fichier. Une autre solution aurait été d'indiquer dans le dataModel de la ressource le type de chaque attribut, pour pouvoir le traiter correctement si besoin, mais ça demande plus de modifications

⚠️ Cette PR est dépendante de assemblee-virtuelle/semapps#1197

@mguihal mguihal force-pushed the fix-DeleteFilesAlongResources branch from 5945b78 to 8b0ab9a Compare November 14, 2023 01:10
@mguihal
Copy link
Collaborator Author

mguihal commented Nov 14, 2023

Pour info: Ajout du commit 8b0ab9a pour mettre à jour Semapps et pouvoir merger cette PR

@srosset81 srosset81 merged commit 691675c into next Nov 14, 2023
@srosset81 srosset81 deleted the fix-DeleteFilesAlongResources branch November 14, 2023 08:28
@mguihal mguihal self-assigned this Nov 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants