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

io.cozy.bank.operations : Stocker les bills de la même manière que les remboursements #33

Open
ptbrowne opened this issue Mar 26, 2018 · 3 comments

Comments

@ptbrowne
Copy link
Contributor

ptbrowne commented Mar 26, 2018

Les remboursements et factures sont stockés dans les io.cozy.bank.operations de cette manière :

var operation = {
  amount: 50,
  reimbursements: [
    { amount: 10, operationId: '123456', billId: '789456' },
    { amount: 20, operationId: '123457', billId: '789457' }
  ],
  bills: [
    'io.cozy.bills:2',
    'io.cozy.bills:3'
  ]
}

Les remboursements et les bills sont stockés de deux manières différentes alors que c'est presque la même chose. On ne peut pas savoir non plus facilement (sans fetcher les factures) la somme des montants des factures d'une opération, par exemple pour savoir si on peut encore rajouter une facture pour cette opération bancaire.

Proposition: stocker les factures de la même manière que les remboursements, avec un array d'objets contenant l'id de la facture et son originalAmount. Le risque de déviation ces données et les données originales de la facture sont très faible vu que le montant d'une facture ne change jamais.

@nono
Copy link
Member

nono commented Mar 26, 2018

Ça pourrait aussi être le bon moment pour commencer à mettre en place la proposition d' @edas sur les relations entre documents

@kosssi
Copy link
Contributor

kosssi commented Mar 26, 2018

👍

surtout que l'on avait dit il y a un moment que mettre 'doctype:id' était inutile ;)

@edas
Copy link
Contributor

edas commented Mar 30, 2018

On n'en est pas encore au point où ça a été validé mais la direction probable est REDACTED

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

No branches or pull requests

4 participants