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

Sprint3/119 tex rates #124

Merged
merged 3 commits into from
Feb 24, 2015
Merged

Sprint3/119 tex rates #124

merged 3 commits into from
Feb 24, 2015

Conversation

aroquemaurel
Copy link
Member

On a une super facture de la mort qui tue et même, que ben elle sait calculer le cout !

Exemple de facture ;)

Super facture

@coveralls
Copy link

Coverage Status

Coverage increased (+0.85%) to 70.97% when pulling 5db771d on sprint3/119-texRates into eecbcfe on S3.

@@ -81,6 +81,8 @@ class Billing : public IModel
*/
void addContributory(Contributory &c);

double getSumRate();
double getSumQuantity();
Copy link
Contributor

Choose a reason for hiding this comment

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

Manque la petite Doxygen qui va bien là :p

Copy link
Member Author

Choose a reason for hiding this comment

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

Ouais… Mais j'aime pas la doxygen :-(

(Mais oui, je vais le faire… >.<)

@manantsoa
Copy link
Contributor

Sinon c'est bien :)
Ah sinon faudra penser à changer les jeux de tests parce qu'avec la nouvelle disposition de l'édition des devis/factures, notamment avec le tableau bah on se retrouve avec des projets qui n'affichent pas leur nom puisqu'on ne peut avoir deux fois un projet dans ce tableau.

great-job

@manantsoa
Copy link
Contributor

Ah j'ai un autre truc à dire, j'ai l'apparition de ce message quand je lance:
QObject::connect: No such slot Gui::MainWindow::openContextupdualMenuTree(const QPoint &) in ../../FactDev/src/mainwindow.cpp:26
QObject::connect: (sender name: 'trCustomers')
QObject::connect: (receiver name: 'MainWindow')

@aroquemaurel aroquemaurel merged commit 5db771d into S3 Feb 24, 2015
aroquemaurel added a commit that referenced this pull request Feb 24, 2015
@aroquemaurel aroquemaurel deleted the sprint3/119-texRates branch November 1, 2017 19:13
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.

None yet

3 participants