entrouvert
/
lingo
16
0
Fork 0
Code Issues Pull Requests 6 Releases Wiki Activity

Tarification: appliquer un taux de réduction au tarif trouvé dans la grille tarifaire (#81231) #100

Merged
lguerin merged 2 commits from wip/81231-pricing-reduction-rate into main 2023-09-21 15:56:05 +02:00
Conversation 2 Commits 2 Files Changed 10 +446 -9
lguerin commented 2023-09-15 14:15:08 +02:00
Owner
Copy Link
No description provided.
lguerin added 1 commit 2023-09-15 14:15:08 +02:00
gitea/lingo/pipeline/head This commit looks good Details
f59d1120b4
pricing: apply a reduction rate (#81231)
lguerin force-pushed wip/81231-pricing-reduction-rate from f59d1120b4 to 71774147ae 2023-09-15 14:33:07 +02:00 Compare
lguerin force-pushed wip/81231-pricing-reduction-rate from 71774147ae to 3199575b84 2023-09-15 15:47:38 +02:00 Compare
lguerin force-pushed wip/81231-pricing-reduction-rate from 3199575b84 to 4c8a8910f5 2023-09-15 15:56:25 +02:00 Compare
lguerin changed title from WIP: Tarification: appliquer un taux de réduction au tarif trouvé dans la grille tarifaire (#81231) to Tarification: appliquer un taux de réduction au tarif trouvé dans la grille tarifaire (#81231) 2023-09-15 15:56:29 +02:00
lguerin force-pushed wip/81231-pricing-reduction-rate from 4c8a8910f5 to 95e6181b64 2023-09-18 14:28:07 +02:00 Compare
lguerin force-pushed wip/81231-pricing-reduction-rate from 95e6181b64 to 3a2ed8d41e 2023-09-18 15:38:49 +02:00 Compare
lguerin force-pushed wip/81231-pricing-reduction-rate from 3a2ed8d41e to c36b18b5ad 2023-09-19 10:17:50 +02:00 Compare
pmarillonnet commented 2023-09-20 15:46:35 +02:00
Owner
Copy Link

Je commence à relire.

Je commence à relire.
pmarillonnet requested review from pmarillonnet 2023-09-20 15:46:43 +02:00
pmarillonnet reviewed 2023-09-20 16:06:14 +02:00
pmarillonnet left a comment
Owner
Copy Link

Questions fonctionnelles pour ma propre compréhension de l’affaire (et donc peut-être est-ce à Cédric qu’il faudrait que je les pose) :

· Mon intuition me dit que dans le cas général on exprimerait la borne minimale (min_pricing) en pourcentage du montant sur lequel on souhaite effectuer la réduction. Comment dans le cas général (où j’imagine le montant sur la grille tarifaire n’est pas connu à l’avance) peut-on exprimer un min_pricing en tant que valeur un peu sortie du chapeau et pas exprimée en fonction du montant de départ ?

· Quel sens fonctionnel donner à max_pricing pour cette PR qui permet l’application de réductions ? Que signifie les cas où le montant après réduction dépasse quand même cette borne supérieure qu’on s’était fixée ?

Questions fonctionnelles pour ma propre compréhension de l’affaire (et donc peut-être est-ce à Cédric qu’il faudrait que je les pose) : · Mon intuition me dit que dans le cas général on exprimerait la borne minimale (`min_pricing`) en pourcentage du montant sur lequel on souhaite effectuer la réduction. Comment dans le cas général (où j’imagine le montant sur la grille tarifaire n’est pas connu à l’avance) peut-on exprimer un `min_pricing` en tant que valeur un peu sortie du chapeau et pas exprimée en fonction du montant de départ ? · Quel sens fonctionnel donner à `max_pricing` pour cette PR qui permet l’application de réductions ? Que signifie les cas où le montant après réduction dépasse quand même cette borne supérieure qu’on s’était fixée ?
lguerin force-pushed wip/81231-pricing-reduction-rate from c36b18b5ad to c260980127 2023-09-21 09:22:49 +02:00 Compare
lguerin commented 2023-09-21 09:53:47 +02:00
Author
Owner
Copy Link

vu avec stefg: on veut bien exprimer min_prixing avec un tarif fixe, ça correspond en tout cas au cas du "pass famille" de vénissieux

et pour le prix plafond, max_pricing, ça n'a pas d'utilité pour le taux de réduction, je vais simplifier

vu avec stefg: on veut bien exprimer min_prixing avec un tarif fixe, ça correspond en tout cas au cas du "pass famille" de vénissieux et pour le prix plafond, max_pricing, ça n'a pas d'utilité pour le taux de réduction, je vais simplifier
lguerin force-pushed wip/81231-pricing-reduction-rate from c260980127 to 580b057bbb 2023-09-21 10:14:37 +02:00 Compare
lguerin force-pushed wip/81231-pricing-reduction-rate from 580b057bbb to 1719655186 2023-09-21 10:19:54 +02:00 Compare
pmarillonnet approved these changes 2023-09-21 15:53:49 +02:00
pmarillonnet left a comment
Owner
Copy Link

Vu tes modifs, top, merci. Ack.

Vu tes modifs, top, merci. Ack.
lguerin changed title from Tarification: appliquer un taux de réduction au tarif trouvé dans la grille tarifaire (#81231) to Tarification: appliquer un taux de réduction au tarif trouvé dans la grille tarifaire (#81231) 2023-09-21 15:55:12 +02:00
lguerin changed target branch from wip/81190-invoicing-invoice-line-payment to main 2023-09-21 15:55:12 +02:00
lguerin merged commit 1719655186 into main 2023-09-21 15:56:05 +02:00
lguerin deleted branch wip/81231-pricing-reduction-rate 2023-09-21 15:56:05 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
pmarillonnet
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/lingo#100
No description provided.
Delete Branch "wip/81231-pricing-reduction-rate"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?