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

Liaison entre les factures et les fichiers #79

Open
kosssi opened this issue Jan 22, 2018 · 3 comments
Open

Liaison entre les factures et les fichiers #79

kosssi opened this issue Jan 22, 2018 · 3 comments
Labels

Comments

@kosssi
Copy link
Contributor

kosssi commented Jan 22, 2018

Problème

Lorsque l'on supprime une facture coté cozy-drive ou cozy-desktop il n'y a pas de suppression du lien dans l'interface de cozy-bank. Ce qui nous retourne une erreur puisque l'on demande à cozy-drive de nous afficher un fichier qui n'existe plus.

Attendu

Ne plus voir afficher le lien coté cozy-bank après avoir supprimé son fichier coté cozy-drive ou cozy-desktop.

Explication

Actuellement on fait un lien entre les factures et les fichiers directement dans un champ de la facture. bill.invoice.

Il faudrait utiliser referenced_by que la stack à mis en place.

Les différents impactes que ça peut avoir coté cozy-bank:

  • l'application va devoir ajouter le doctype io.cozy.files dans ses autorisations.
  • modifier l'import des factures
  • faire une migration
@kosssi kosssi added the bug label Jan 22, 2018
@doubleface
Copy link
Collaborator

Est-ce que drive gère bien le fait qu'un fichier soit référencé (affichage du bon message d'erreur ou mise à jour en cascade des documents qui le référencent) ? Ce n'était pas le cas à l'époque où on avait voulu les utiliser. @zoe avait je crois remonté le problème au appvengers mais je ne sais pas ce qu'il s'est passé par la suite sur drive et desktop.

Il y a aussi le problème de l'application desktop qui va avoir peut-être de nouveaux cas d'erreur à gérer (impossible de modifier, supprimer un fichier)

@ptbrowne
Copy link
Contributor

ptbrowne commented Feb 13, 2018

See this related issue : cozy/cozy-stack#1189 (comment)

@nono is advising to add the file to the referenced_by of the bill.

@ptbrowne
Copy link
Contributor

l'application va devoir ajouter le doctype io.cozy.files dans ses autorisations.

Pourquoi ? Je pense qu'on doit avoir le lien dans les deux sens.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants