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

PDF INVOICE : Wrong title for $REF, add line for $INVOICE_REF #2242

Open
wants to merge 3 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@InformatiqueProg
Contributor

InformatiqueProg commented Aug 18, 2016

No description provided.

@roadster31

This comment has been minimized.

Contributor

roadster31 commented Oct 31, 2017

En France, il faut que les numéros de facture soient consécutifs.
En utilisant la ref de la commande comme numéro de facture, on ne respecte pas ce principe.

@roadster31 roadster31 added the question label Oct 31, 2017

@InformatiqueProg

This comment has been minimized.

Contributor

InformatiqueProg commented Oct 31, 2017

C'est bien pour ça que j'utilise systématiquement le module InvoiceRef
https://github.com/thelia-modules/InvoiceRef

@roadster31

This comment has been minimized.

Contributor

roadster31 commented Oct 31, 2017

Certes. Mais quelle solution pour le core ?

@InformatiqueProg

This comment has been minimized.

Contributor

InformatiqueProg commented Oct 31, 2017

Maintenant pourquoi pas utiliser le fonctionnement "à la norme Française" de base dans Thélia ?
et laisser aux modules la possibilité de modifier ça (via un event par exemple)

@roadster31

This comment has been minimized.

Contributor

roadster31 commented Oct 31, 2017

Genre intégrer le module InvoiceRef au core ?

@InformatiqueProg

This comment has been minimized.

Contributor

InformatiqueProg commented Oct 31, 2017

Bonne idée !
cela permet de limiter les problèmes de mise à jour pour ceux (comme moi) qui utilise déjà ce module

@roadster31

This comment has been minimized.

Contributor

roadster31 commented Oct 31, 2017

C'est vrai. Tu veux faire ça dans cette PR ?

@InformatiqueProg

This comment has been minimized.

Contributor

InformatiqueProg commented Oct 31, 2017

Si tu es d'accord pour cette solution, je peux m'en occuper (d'ici quelques jours)

@roadster31

This comment has been minimized.

Contributor

roadster31 commented Oct 31, 2017

Deal !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment