Document with filefield

[rufener]

Ce PR redéfinit le modèle Document
Les objets document sont poussés juste après avoir enregistré l'objet item.
⚠️ La suppression de documents se fait "on the fly", sans avoir besoin d'enregistrer l'item. Le message d'alerte avant d'effacer le document est spécifique au document en question (affiche le filename).

[maltaesousa]

I've reviewed backend, I'll let @mparkan review the front part :)

[rufener]

⚠️ Le chemin d'enregistrement des fichiers est PurePath(str(item.created.year), item.number, file.name). Le problème est que le champ item.number est un champs modifiable dans le formulaire. Est-ce qu'on rend ce champ immuable ou est-ce qu'on prend l'identifiant de l'item (item.id) pour remédier à un changement de dossier qui briserait les liens vers les documents ?

L'avantage de item.number est qu'il est très facile à retrouver la logique de quel OP on parle, si d'aventure un jour on devait aller sur le serveur pour une raison X ou Y.

L'avantage de item.id est qu'il est immuable.

[rufener]

⚠️ Le chemin d'enregistrement des fichiers est PurePath(str(item.created.year), item.number, file.name). Le problème est que le champ item.number est un champs modifiable dans le formulaire. Est-ce qu'on rend ce champ immuable ou est-ce qu'on prend l'identifiant de l'item (item.id) pour remédier à un changement de dossier qui briserait les liens vers les documents ?

L'avantage de item.number est qu'il est très facile à retrouver la logique de quel OP on parle, si d'aventure un jour on devait aller sur le serveur pour une raison X ou Y.

L'avantage de item.id est qu'il est immuable.

Je passe sur l'id parce que le champ number n'a pas de contrainte sur les signes autorisés. Il pourrait y avoir des conflits avec les caractères dans le path.

[maltaesousa]

Ah ok, pour les prochaines draft, ajoute les reviewers seulement quand la PR est prête :) je pensais que tu voulais la review maintenant

[rufener]

Ah ok, pour les prochaines draft, ajoute les reviewers seulement quand la PR est prête :) je pensais que tu voulais la review maintenant

Hmmm curieux, je n'ai pas souvenir avoir déjà enregistré les reviewers... Bref, je vous ferai signe quand le PR est prêt 😉

[mparkan]

LGTM (Still have to check issue with uploaded documents only appearing after a page refresh)