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

Sprint6/155 daily hours user #238

Merged
merged 7 commits into from
Apr 8, 2015
Merged

Sprint6/155 daily hours user #238

merged 7 commits into from
Apr 8, 2015

Conversation

aroquemaurel
Copy link
Member

Renseignement de l'utilisateur des informations suivantes

  • Nombre d'heure par jour
  • Nombre de jours travaillé par semaine
  • Nombre de jours travaillé par mois.

Sauvegarde de ces données, utilisation dans les chiffrage.

double _nbHoursPerDay;
double _nbDaysPerWeek;
double _nbDaysPerMonth;

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 documentation des attributs

@coveralls
Copy link

Coverage Status

Coverage increased (+0.12%) to 87.78% when pulling c409931 on sprint6/155-dailyHoursUser into 30b6143 on S6.

@KraTuX31
Copy link
Contributor

KraTuX31 commented Apr 8, 2015

lorsqu'on créé un devis ou une facture, lorsqu'on saisi le tarif horaire du projet, cela ne modifie par le tarif journalier en fonction de celui saisi dans les paramètres

@aroquemaurel
Copy link
Member Author

Woops

@KraTuX31
Copy link
Contributor

KraTuX31 commented Apr 8, 2015

remarque aussi, la combobox pour choisir l'unité (heures/jours/mois) ne semble plus fonctionner...

@aroquemaurel
Copy link
Member Author

On doit être bon maintenant ;-)

@KraTuX31
Copy link
Contributor

KraTuX31 commented Apr 8, 2015

Ok j'ai "check" quand Travis te dit go...ben go tu peux merge^^

@coveralls
Copy link

Coverage Status

Coverage increased (+0.13%) to 87.79% when pulling 8647fbc on sprint6/155-dailyHoursUser into 30b6143 on S6.

@aroquemaurel
Copy link
Member Author

Yeah ! :-)

@coveralls
Copy link

Coverage Status

Coverage increased (+0.13%) to 87.79% when pulling 9e63bf7 on sprint6/155-dailyHoursUser into 30b6143 on S6.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.13%) to 87.79% when pulling 9e63bf7 on sprint6/155-dailyHoursUser into 30b6143 on S6.

aroquemaurel added a commit that referenced this pull request Apr 8, 2015
@aroquemaurel aroquemaurel merged commit 376ee1c into S6 Apr 8, 2015
@aroquemaurel aroquemaurel deleted the sprint6/155-dailyHoursUser 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