cozy / cozy-doctypes

The doctypes used in cozy-stack
https://docs.cozy.io/en/cozy-doctypes/docs/
GNU Affero General Public License v3.0
2 stars 29 forks source link

Update bank doctypes #26

Closed kosssi closed 6 years ago

kosssi commented 6 years ago


ptbrowne commented 6 years ago

Si un tableau contient des ids, je préférais qu'on rajoute ids à la fin : billIds, invoice : invoiceIds. Pourquoi un split entre creditOperations et debitOperations ? Qu'est ce que ca apporte ? C'est le même doctype non ? Parce qu'après on pourrait se dire pourquoi ne pas mettre healthDebitOperations, transportDebitOperations ?

doubleface commented 6 years ago

ajout des liens vers les opérations crédits débits

  • creditOperations
  • debitOperations

Ca veut dire qu'on veut doubler les références? Une opération de débit référence les opérations de crédit associée et inversement ?

Ca risque d'être une galère à maintenir, non?

doubleface commented 6 years ago

Et on ne spécifie donc plus le doctype associé à l'id? Ca veut dire qu'on récupère d'abord le document associé à l'id avant de voir quelle action on peut faire dessus du coup.

kosssi commented 6 years ago

@doubleface on n'a pas vraiment besoin de spécifier le doctype puisqu'on le connait. Ce serait vraiment bizarre d'avoir des bills qui sont des io.cozy.files vu que l'on décide, il faut se mettre d'accord, ça simplifie la logique ainsi que le code des applications.

Ok pour gérer qu'un seul lien (operationsIds? vous voyez un autres nom plus explicite ?) et mettre Ids dans le nom, je vais changer le descriptif de la PR.

nono commented 6 years ago

@kosssi @ptbrowne @doubleface ça donne quoi cette PR ?

ptbrowne commented 6 years ago

Merci @nono d'avoir pingé.