Skip to content

Sprint4/134 period turnover#185

Merged
aroquemaurel merged 13 commits into
S4from
sprint4/134-periodTurnover
Mar 18, 2015
Merged

Sprint4/134 period turnover#185
aroquemaurel merged 13 commits into
S4from
sprint4/134-periodTurnover

Conversation

@manantsoa
Copy link
Copy Markdown
Contributor

Possibilité de calculer le chiffre d'affaires sur une période choisie par l'utilisateur

  • Ajout d'un menu dans la barre de menu "Statistiques" pour tout ce qui concernera les chiffres pouvant intéresser l'utilisateur (discret, il n'embêtera personne)
  • Ajout d'une fenêtre pouvant calculer le chiffre d'affaires entre 2 dates, avec le nombre de factures trouvées dans cet intervalle

Note: Si vous avez d'autres idées de fonctionnalités à rajouter dans cette nouvelle fenêtre, on peut en parler ;)

@manantsoa manantsoa added this to the Sprint 4 milestone Mar 17, 2015
@KraTuX31
Copy link
Copy Markdown
Contributor

Au niveau de la lisibilité et documentation c'est niquel. Je teste voir

@KraTuX31
Copy link
Copy Markdown
Contributor

  • Alors à mon goût l'IHM est vraiment pas top. Champ un peu disposer n'importe comment. La fenêtre n'a pas de taille min/max défini, ni rien de responsive...bref c'est pas top. Perso j'aurais vu plus quelque chose comme ça:

capture-calcul du chiffre d affaires

  • Ensuite, sur l'aspect fonctionnel, par défaut je mettrait comme date de début le 01/01/AnneeCourante et date de fin dateDuJour/MoisCourant/AnneeCourante

@manantsoa
Copy link
Copy Markdown
Contributor Author

Okay, je prends en compte tes remarques pertinentes.

@manantsoa
Copy link
Copy Markdown
Contributor Author

Monsieur est servi :)

@KraTuX31
Copy link
Copy Markdown
Contributor

Je parle aussi en la personne d'Antoine.

  • en fait au niveau IHM, utiliser des calendars directement (plus joli)
    capture-calcul du chiffre d affaires - previsualisation
  • Faudrait que si une date de début est supérieure à celle de fin ben ça inverse
  • Il faut que ça prennent uniquement les Factures PAYÉES!
    --> Dans ta requête SQL "isPaid = 1"

@aroquemaurel
Copy link
Copy Markdown
Member

Je plussoie, l'idée des Calendar me semble excellente ! =)

@manantsoa
Copy link
Copy Markdown
Contributor Author

Bon voilà j'espère avoir répondu à toutes vos demandes XD
Par contre juste pour l'inversion des dates du calendrier lorsque la date de début ou de fin dépasse l'autre date je l'ai pas fait parce que je suis pas d'accord.
Pour moi, l'utilisateur sait qu'il est en train de choisir sa date de début de période, donc s'il veut choisir une date de période qui est supérieure à la date de fin, en sachant qu'il veut modifier la date de fin ensuite, et que les deux dates s'inversent, moi je trouve ça plus embêtant qu'autre chose ^^ ça va le faire plus galèrer qu'autre chose alors qu'en mettant juste la date de fin égale à celle de début lorsqu'il y a dépassement je pense que ça l'aidera largement plus :)

@KraTuX31
Copy link
Copy Markdown
Contributor

Le calcul me paraît foireux, obtenir 621. Perso quand je regarde dans la BD, je vois que y'a que la facture 1 qui est a été payé. Celle-ci est associé à trois projets (23, 42, 28) ayant respectivement ces tarifs horaires 12.42, 15.58, 16.8 . Si on fait * 7 (nombre d'heures par jours, on obtient 313,642).
Je pense que y'a une couille après je vois pas trop où...

@Oxynos
Copy link
Copy Markdown
Member

Oxynos commented Mar 18, 2015

Actuellement on gère toutes les factures de la même façon, il faudrait ouvrir une issue sur le prochain sprint si on veut gérer le fait de calculer le chiffre d'affaire que en fonction des factures payés, c'est pas très compliqué mais ça veut dire qu'il faudrait aussi définir si le coût d'un projet est en fonction des factures payés ou pas ... (à définir du coup)

@aroquemaurel
Copy link
Copy Markdown
Member

Le coup des projets est pas bon (dans le tableau).

  • Projet Nun ac → 141.25€
    Or dans ce projet, y'a deux factures qui ne font clairement pas ce montant là…
    Je vais vérifier, ça serait bien d'avoir un truc qui marche pour demain, mais ça ne m'etonnerait pas que ce problème soit lié à celui de Tsoo Tsoo…

Ce que je propose, je sais c'est mal, vous le savez, mais ça simplifiera beaucoup de choses je pense.
@manantsoa : Tu intègres. (Tant pis pour les problèmes)
moi, je fini ma pull request, puis j'intègre.
anyone, fini le Beamer : Juste la conclu + des screens de FactDev (présents dans le user manual)
anyone fini le user manual (ajouter la feature de Tsoo avec de jolis screens)

Une fois qu'on a fait tout ça, on résoudra les problèmes ;)

PSSST : Nous sommes à 10 messages sur une PR, sans connerie, sans petits chats, juste à parler de boulot o_Ô

@Oxynos
Copy link
Copy Markdown
Member

Oxynos commented Mar 18, 2015

Je finis le beamer avec les screens

@KraTuX31
Copy link
Copy Markdown
Contributor

je finis le user manual

@aroquemaurel
Copy link
Copy Markdown
Member

J'intègre cette PR ;-)

@manantsoa
Copy link
Copy Markdown
Contributor Author

Il y a effectivement un problème quelque part mais où :)
Néanmoins ce n'est pas dans le coût du projet ^^ Si tu fais la somme de chaque coût du projet Nunc ac dans chaque facture on obtient le bon prix. C'est un peu bordélique mais ça marche XD
Faudra déjà enlever les factures non-payées du calcul du coût mais là pour l'instant je vois pas du tout le problème :/

@aroquemaurel aroquemaurel merged commit 0499ef0 into S4 Mar 18, 2015
aroquemaurel added a commit that referenced this pull request Mar 18, 2015
…Team/FactDev into S4

Conflicts:
	fact-team.github.io
@aroquemaurel
Copy link
Copy Markdown
Member


Du coup j'ai déjà mergé, j'espère qu'on se tape pas dessus :-°

@manantsoa
Copy link
Copy Markdown
Contributor Author

Si je vais te taper :p j'allais merger je cherchais juste l'erreur ^^ Mais bon pas grave XP

@aroquemaurel
Copy link
Copy Markdown
Member

:-p

@aroquemaurel aroquemaurel deleted the sprint4/134-periodTurnover branch November 1, 2017 19:14
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.

4 participants