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

Tickets #414

Closed
wants to merge 10 commits into from
Closed

Tickets #414

wants to merge 10 commits into from

Conversation

jean140950
Copy link
Contributor

Modifications pour imprimer des tickets incluant les informations du vendeur et le détail des articles de la vente

@darnuria
Copy link
Contributor

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)} €`;
Copy link
Contributor

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.

@jean140950
Copy link
Contributor Author

jean140950 commented Nov 16, 2020 via email

@darnuria
Copy link
Contributor

darnuria commented Nov 18, 2020

@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.
Cependant je vais totalement reprendre les modifications que tu propose pour les tickets et dans une autre intégration mettre a jour bootstrap'.

@darnuria
Copy link
Contributor

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 master.

Ensuite peut-être je te demanderais de découper ou de motiver tes autres changements car la c'est un peu compliqué a comprendre.

@jean140950
Copy link
Contributor Author

@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 n'ai eu aucune action sur les autres fichiers modifiés, j'ai récupéré ces changements en clonant oressource et j'ai pensé que c'était des modifications faites par d'autres ou automatiques, elles portent sur morris, bootstrap, jquery, raphael...
Le seul module concerné par un acte de vente est bootstrap-switch.js que je n'ai pas modifié bien qu'il soit mal interprété par Chromium
Fais les modifications qui te paraissent à faire, incluant l'impression des tickets, si ça te convient,
En ce quinous concerne nous nous alignerons sur le master final sur mart1ver lorsque tu nous donnera ton feu vert
Bon courage

@darnuria
Copy link
Contributor

Je viens de comprendre ya eu un bins ! Un gros renomage de fichiers involontaire!
Je régle ça,

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! ^_^

@darnuria
Copy link
Contributor

darnuria commented Nov 18, 2020

PS git c'est retord comme système faut vraiment faire attention a ce que on git add car si on fait git add -a ou git add *, il est pas malin il fait exactement ce qu'on demande donc si on bouge involontairement du code il sabre tout :D

@darnuria
Copy link
Contributor

Pour suivre voici les petites sous-pr que j'ai faites pour suivre plus raisonnablement:

@darnuria
Copy link
Contributor

darnuria commented Nov 18, 2020

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!
C'est top, on va ajouter bout par bout assez vite j'aime pas ajouter des gros bouts pense a git checkout master si tu veux refaire une PR sur un autre sujet! :))

@darnuria darnuria closed this Nov 18, 2020
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

Successfully merging this pull request may close these issues.

2 participants