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

Vue « comptabilité » + export CSV #110

Merged
merged 19 commits into from Mar 2, 2018

Conversation

Projects
None yet
6 participants
@JocelynDelalande

JocelynDelalande commented Feb 22, 2018

Concerne la demande #32

Pour l'instant juste une version de travail basique. ne pas fusionner maintenant.

En l'état, ça ressemble à ça:

image

Reste à faire:

  • filtrage possible pour n'afficher qu'une partie des factures (ex: par date)
  • export CSV (cf #89 )

@JocelynDelalande JocelynDelalande changed the title from WIP: Vue « comptabilité » to Vue « comptabilité » + export CSV Feb 23, 2018

@JocelynDelalande

This comment has been minimized.

JocelynDelalande commented Feb 23, 2018

Ça me semble prêt concernant les deux points qui restaient à traiter ; quelques captures d'écran pour se mettre l'eau à la bouche ;-)

image

image

image

@JocelynDelalande

This comment has been minimized.

JocelynDelalande commented Feb 23, 2018

Par contre question métier : dans l'export CSV anonymisé (ndlr: plutôt destiné au comptable, dans l'idée), j'ai gardé les champs suivants:

  • date
  • n° de facture
  • montant
  • moyen de paiement
  • devise

En manque-t-il ?

@littlejo

This comment has been minimized.

Collaborator

littlejo commented Feb 23, 2018

Bravo ! C'est prometteur !

@moullec

This comment has been minimized.

Collaborator

moullec commented Feb 24, 2018

Peut-être un numéro de patient en plus du numéro de facture ?

@JocelynDelalande

This comment has been minimized.

JocelynDelalande commented Feb 24, 2018

Peut-être un numéro de patient en plus du numéro de facture ?

La question est : est-ce que le comptable en a besoin ? Si non, moins on en met dans cette vue, mieux c'est (vie privée).

@moullec @jkiwi @nicosteo @Polyne85 (et autres praticien·e·s), un avis sur la question ? un avis venant de votre comptable ?

JocelynDelalande added some commits Feb 22, 2018

Cleanup imports according to pep-8
- All imports at top of file
- 3 alphabetically ordered groups imports
- Remove numerous unused imports

See https://www.python.org/dev/peps/pep-0008/#imports

iso-functional commit.
Offer period selection interface in billing view
- defaults to current month
- can select two dates from a Widget
- can select predefined periods : current month, last year, previous year
Provide localization for daterangepicker
- localization mechanism (laying on django's built-in stuff)
- french translation
Fix indentation
This was 4-spaces everywhere but 1-tab here… No longer :-)

(iso-functional commit)
Allow client to choose invoice CSV fields
Via `fields` querystring on endpoint invoice listing URL.
@JocelynDelalande

This comment has been minimized.

JocelynDelalande commented Feb 24, 2018

Conflits corrigés.

@opi

This comment has been minimized.

Contributor

opi commented Feb 25, 2018

Ajouter le statut de la facture ?

@JocelynDelalande

This comment has been minimized.

JocelynDelalande commented Feb 25, 2018

Ajouter le statut de la facture ?

Ça se tient :-)

@jkiwi

This comment has been minimized.

jkiwi commented Feb 26, 2018

@moullec @JocelynDelalande
Même réflexion que Jocelyn. J'ai cherché mais je pense que le numéro du patient n'est utile qu'au fonctionnement du logiciel, donc pas forcément besoin de l'afficher ici sur la vue compta. C'est très peu utile pour le comptable et voir même pour le praticien (sauf peut-être en cas de maintenance ?...)

@moullec

This comment has been minimized.

Collaborator

moullec commented Feb 26, 2018

@jkiwi @JocelynDelalande J'y pensais pour les cas où les patients perdent leur facture, afin de faciliter la réédition, mais effectivement ce ne semble pas indispensable pour l'expert comptable

@jkiwi

This comment has been minimized.

jkiwi commented Feb 26, 2018

@moullec ah ok. Certe ça peut arriver :) une recherche avec simplement leur nom devrai pouvoir faire ressortir la ligne correspondante pour une réédition ? (que ce soit dans un tableau avec l'ensemble des patients ou dans le tableau compta ?)

@moullec

This comment has been minimized.

Collaborator

moullec commented Feb 26, 2018

@jkiwi effectivement avec une recherche nominative on doit pouvoir retrouver la facture perdue (ou les pour certains patients..)

@jbgury

jbgury approved these changes Mar 2, 2018

@jbgury

This comment has been minimized.

Contributor

jbgury commented Mar 2, 2018

Je peux merger ?

Merge pull request #98 from opi/installation-forms
Improve installation forms style (restore and register).
@JocelynDelalande

This comment has been minimized.

JocelynDelalande commented Mar 2, 2018

@jbgury avec grand plaisir :-)

@jbgury

This comment has been minimized.

Contributor

jbgury commented Mar 2, 2018

👍
Quelques remarques mineures :

  • l'action imprimer devrait ouvrir dans un nouvel onglet
  • "Payée" au lieu de "Payé" :)

@jbgury jbgury merged commit 56e0ec5 into libreosteo:develop Mar 2, 2018

This was referenced Mar 2, 2018

@JocelynDelalande JocelynDelalande deleted the JocelynDelalande:jd-invoices-list branch Mar 2, 2018

@JocelynDelalande

This comment has been minimized.

JocelynDelalande commented Mar 2, 2018

@jbgury OK, merci d'avoir toi même corrigé tes petites remarques du coup, gentil de ta part :)

jbgury added a commit that referenced this pull request Mar 3, 2018

Merge pull request #123 from JocelynDelalande/jd-remove-render-to-res…
…ponse

Replace render_to_response() calls with render() (reprise de #110)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment