-
Notifications
You must be signed in to change notification settings - Fork 17
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
Tickets #414
Tickets #414
Conversation
…ion.php repertoire core
Wow c'est toujours agréable bravo! 💯 Toute contribution est la bienvenue je prends le temps de regarder d'ici ce wekend. |
@@ -432,9 +433,15 @@ function add() { | |||
* @returns {string} Representant un fragment HTML. | |||
*/ | |||
show() { | |||
const prixTxt = `${this.prix} €`; | |||
const prixTxt = `${this.prix.toFixed(2)} €`; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bien vu. Dans un futur radieux on devrait eviter l'usage des nombres basiques de javascript car ce sont des nombres en virgule flottante... Un jour il faudrait s'embêter a utiliser une bibliothèque qui permettre la gestion saine des valeurs monetaires.
Merci Axel,
Par contre je pense qu’il y a des bugs dans ce que j’ai envoyé
Je suis en cours de recherche
Je posterai les modifs si je les trouve dans github
Jean
[Axel: edit pour retirer les informations personnelles/quote du courriel]
|
@jean140950 j'apprecie énormément ton implication mais on va pas trop toucher a comment sont organisé les sources. C'est une histoire de maintenance. Je peux accepter des patchs, des fonctionnalités mais tout changer comme 59 fichiers ça commence à être complexe à revérifier. Je te propose qu'on intégré déjà les fix super pertinents de l'impression et dans d'autres pull-request (demande d'intégration) peut-être ajouter tes modifications qui je pense seront pertinentes. :) @mart1ver essaie de suivre aussi c'est un commun on peut pas tout casser on à déjà eu le soucis (par ma faute!), et martin a eu beaucoup de mal a suivre après une grosse phase de développement certes utile mais qui a vraiment changer les sources. Excuse moi de faire mon rabat joie mais au final c'est martin et moi qui maintenons et on a pas un temps infini. |
Pour la procédure: Je vais récupérer ta branche localement tester pour les tickets, peut-être faire une ou deux modif si je trouve un bug et ensuite intégrer dans Ensuite peut-être je te demanderais de découper ou de motiver tes autres changements car la c'est un peu compliqué a comprendre. |
@darnuria les modifs que j'ai faites portent sur le nouveau fichier css, qui décrit le style à appliquer au ticket- plus facile à modifier si on veut modifier le rendu du ticket. les autres fichiers concernés sont utils.js, ventes.js et ventes.php. |
Je viens de comprendre ya eu un bins ! Un gros renomage de fichiers involontaire! J'te corrige tes petits bugs et j’intègre doucement c'est du bon boulot je fait passer les petits fix en priorité! Juste attention ici on code l'indentation en espaces! ^_^ |
PS git c'est retord comme système faut vraiment faire attention a ce que on |
Pour suivre voici les petites sous-pr que j'ai faites pour suivre plus raisonnablement:
|
Hop découpage fait je close ici et je laisse 2~3j a martin pour voir si il peut relire sinon je merge samedi environ! |
Modifications pour imprimer des tickets incluant les informations du vendeur et le détail des articles de la vente