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

Sprint4/139 fork quote billing #165

Merged
merged 16 commits into from
Mar 9, 2015
Merged

Sprint4/139 fork quote billing #165

merged 16 commits into from
Mar 9, 2015

Conversation

manantsoa
Copy link
Contributor

Remarque: pas d'icône pour la copie d'un devis ou d'une facture :/
Autre remarque: pas de Doxygen dans la classe ContributoriesList :(

@Oxynos Oxynos added this to the Sprint 4 milestone Mar 8, 2015
@aroquemaurel
Copy link
Member

@KraTuX31 : pour les icons de copie de devis, tu sais ce qu'il te reste à faire ;-)

@manantsoa, tu dois pouvoir faire un tests ou deux pour pouvoir augmenter la couverture ;)

@@ -70,6 +90,16 @@ void AddQuoteDialog::reject() {
QDialog::reject();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On pourrait pas péter cette méthode ? Ça commence à être un peu gros :p

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je vais voir ce que je peux faire ;)

@aroquemaurel
Copy link
Member

  • Niveau code, pas grand chose à redire
  • Niveau utilisation, ça marche bien
  • Par contre, j'ajouterai bien une petite feature, c'est que quand tu copie un devis, tu ai la possibilité de la transformer en facture et vis-versa (en gros, juste changer le isBilling)
    Tu peux faire un bouton « Transformer en Devis/facture » en fonction du contexte ?
    Comme tu le sens

@manantsoa
Copy link
Contributor Author

Ouais c'est pas une mauvaise idée, je vais faire ça ;) Par contre niveau IHM ça donnerait quoi ? Dans la fenêtre du devis ou facture je mets un bouton en plus quand je fais une copie qui transforme la facture en devis et inversement ?

@aroquemaurel
Copy link
Member

Deux solutions pour moi :

  • Soit un bouton copier, et un autre bouton qui copie + transforme en…
  • Soit un bouton copier, avec en + à côtés du bouton une liste déroulante contenant Facture ou Devis.
    Donc, bouton copier + liste déroulante sélectionné Facture → On copie en facture.
    Par défaut, la liste déroulante aurait un currentIndex = au type du billing. (Facture → on préselecitonne facture, Devis on préselectionn Devis)

À toi de voir ce que tu préfères =)

@manantsoa
Copy link
Contributor Author

Ah bah j'avais pas vu ton commentaire du coup j'ai fait un peu comme je pensais :) En fait quand on copie un devis ou une facture j'ai ajouté un bouton changer en devis/facture en bas à gauche de la fenêtre, par contre il s'affiche pas pour un nouveau ou pour l'édition :)

@aroquemaurel
Copy link
Member

Yep, c'est cool comme ça aussi :)

Juste, quand tu change en Facture, à ce moment là peut être modifier le texte du bouton, pour qu'il devienne changer en Devis et vis-versa, non ? =)
(et ça manque d'icon :p Ptêtre mettre juste l'icon de devis quand c'est changer en devis et juste l'icon de facture quand c'est changer en facture)

Par contre, il faut faire gaffe au niveau de la numérotation si on change devis/facture je pense, je sais pas si c'est bien gété ça ? =)

@manantsoa
Copy link
Contributor Author

Pour le bouton j'avais oublié en fait ^^' sinon normalement les numéros des factures/devis se changent bien en fonction de leur type dans changeDocType tu peux tester si tu veux ça marche :p

Du coup bah c'est bon je crois, je ferais encore un peu de refactoring et changer les trucs qui manquent et pis j'aurais le droit de merger je crois ^^

@aroquemaurel
Copy link
Member

Ça me va =)

@manantsoa
Copy link
Contributor Author

Bon j'ai fait un peu de refactoring et j'ai rajouté les icônes, je peux merger ? :)

@aroquemaurel
Copy link
Member

Yep ;-)

@manantsoa manantsoa merged commit 559d882 into S4 Mar 9, 2015
manantsoa added a commit that referenced this pull request Mar 9, 2015
…T-Team/FactDev into S4

Conflicts:
	src/gui/mainwindow/mainwindow.cpp
	src/gui/mainwindow/mainwindow.ui
	src/icons.qrc
@aroquemaurel aroquemaurel deleted the sprint4/139-ForkQuoteBilling branch November 1, 2017 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants