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

tarification: taux de réduction au lieu d'un tarif minimum unique, on a besoin de matrices de tarif minimum (#89605) #186

Merged
lguerin merged 6 commits from wip/89605-pricing-reduction-min-price into main 2024-04-26 10:46:04 +02:00
Conversation 2 Commits 6 Files Changed 21 +1326 -662
lguerin commented 2024-04-17 16:45:04 +02:00
Owner
Copy Link
No description provided.
lguerin added 5 commits 2024-04-17 16:45:04 +02:00
lguerin force-pushed wip/89605-pricing-reduction-min-price from 09005b9613 to 8678ec90dc 2024-04-18 10:39:23 +02:00 Compare
lguerin changed title from WIP: tarification: taux de réduction au lieu d'un tarif minimum unique, on a besoin de matrices de tarif minimum (#89605) to tarification: taux de réduction au lieu d'un tarif minimum unique, on a besoin de matrices de tarif minimum (#89605) 2024-04-18 14:48:00 +02:00
pmarillonnet requested review from pmarillonnet 2024-04-18 15:38:32 +02:00
pmarillonnet commented 2024-04-18 15:38:40 +02:00
Owner
Copy Link

(Je commence à relire.)

(Je commence à relire.)
pmarillonnet reviewed 2024-04-22 15:29:54 +02:00
pmarillonnet left a comment
Owner
Copy Link

Un premier bout de relecture, surtout des remarques de forme, dis-moi ce que tu en penses, je referai un tour ensuite.

Un premier bout de relecture, surtout des remarques de forme, dis-moi ce que tu en penses, je referai un tour ensuite.
lingo/pricing/models.py Outdated
@ -585,3 +595,3 @@
return self.get_extra_variables(request, context)
def format_pricing_data(self):
def format_pricing_data(self, field):
pmarillonnet commented 2024-04-22 15:27:37 +02:00
Owner
Copy Link

Par principe je me disais peut-être mettre une valeur par défaut field='pricing' pour ce nouveau paramètre histoire de ne pas casser l’interface existante (?)

Par principe je me disais peut-être mettre une valeur par défaut `field='pricing'` pour ce nouveau paramètre histoire de ne pas casser l’interface existante (?)
lguerin commented 2024-04-22 15:36:55 +02:00
Author
Owner
Copy Link

ok, j'ai ajouté un fixup

ok, j'ai ajouté un fixup
pmarillonnet commented 2024-04-23 08:58:22 +02:00
Owner
Copy Link

Ok top.

Ok top.
pmarillonnet marked this conversation as resolved
lingo/pricing/templates/lingo/pricing/manager_pricing_agenda_form.html
@ -21,2 +21,4 @@
</form>
{% endblock %}
{% block sidebar %}
pmarillonnet commented 2024-04-22 14:16:13 +02:00
Owner
Copy Link

Question d’UI, à relire l’héritage de gabarits j’ai l’impression qu’on ajoute une barre latérale vide à chaque fois, ici et dans les gabarits suivants, je loupe un truc ou bien c’est volontaire que d’avoir hérité de manager_pricing_parameters pour ensuite effacer le contenu de la barre latérale qui y était défini ?

Question d’UI, à relire l’héritage de gabarits j’ai l’impression qu’on ajoute une barre latérale vide à chaque fois, ici et dans les gabarits suivants, je loupe un truc ou bien c’est volontaire que d’avoir hérité de manager_pricing_parameters pour ensuite effacer le contenu de la barre latérale qui y était défini ?
lguerin commented 2024-04-22 15:35:03 +02:00
Author
Owner
Copy Link

L'héritage c'est surtout pour avoir les breadcrumbs et pouvoir en ajouter un.

L'héritage c'est surtout pour avoir les breadcrumbs et pouvoir en ajouter un.
pmarillonnet commented 2024-04-23 08:58:46 +02:00
Owner
Copy Link

Ah ok j’avais loupé ça, merci.

Ah ok j’avais loupé ça, merci.
pmarillonnet marked this conversation as resolved
lingo/pricing/templates/lingo/pricing/manager_pricing_detail.html Outdated
@ -48,0 +20,4 @@
<button aria-controls="panel-matrix" aria-selected="true" id="tab-matrix" role="tab" tabindex="0">{% trans "Pricings" context 'amount' %}</button>
{% if object.kind == 'effort' %}
<button aria-controls="panel-pricing-options" aria-selected="false" id="tab-pricing-options" role="tab" tabindex="-1">{% trans "Pricing options" %}</button>
{% endif %}
pmarillonnet commented 2024-04-22 15:09:15 +02:00
Owner
Copy Link

(Juste in elif ici pour gagner une ligne ? 🙂)

(Juste in elif ici pour gagner une ligne ? 🙂)
lguerin commented 2024-04-22 15:36:44 +02:00
Author
Owner
Copy Link

yep, fixup :)

yep, fixup :)
pmarillonnet commented 2024-04-23 08:59:15 +02:00
Owner
Copy Link

Top.

Top.
pmarillonnet marked this conversation as resolved
lingo/pricing/templates/lingo/pricing/manager_pricing_detail.html Outdated
@ -275,0 +77,4 @@
<ul>
<li>{% trans "Minimal pricing:" %} {{ object.min_pricing|default_if_none:"" }}</li>
</ul>
{% if object.kind == 'effort' %}
pmarillonnet commented 2024-04-22 15:13:25 +02:00
Owner
Copy Link

Un if pas nécessaire ici, il reprend exactement la même condition que le if du bloc parent.

Un if pas nécessaire ici, il reprend exactement la même condition que le if du bloc parent.
lguerin commented 2024-04-22 15:37:39 +02:00
Author
Owner
Copy Link

en effet, fixup

en effet, fixup
pmarillonnet marked this conversation as resolved
lingo/pricing/templates/lingo/pricing/manager_pricing_detail.html Outdated
@ -275,0 +86,4 @@
<a class="pk-button" rel="popup" href="{% url 'lingo-manager-pricing-pricingoptions-edit' object.pk %}">{% trans "Edit pricing options" %}</a>
</div>
</div>
{% endif %}
pmarillonnet commented 2024-04-22 15:14:00 +02:00
Owner
Copy Link

(Pareil ici peut-être juste un elif car les conditions sont mutuellement exclues ?)

(Pareil ici peut-être juste un elif car les conditions sont mutuellement exclues ?)
lguerin commented 2024-04-22 15:38:30 +02:00
Author
Owner
Copy Link

fixup aussi

fixup aussi
pmarillonnet marked this conversation as resolved
lingo/pricing/templates/lingo/pricing/manager_pricing_matrix_form.html Outdated
@ -5,3 +5,3 @@
{{ block.super }}
{% if matrix.criteria %}
<a href="{% url 'lingo-manager-pricing-matrix-slug-edit' object.pk matrix.criteria.slug %}">{% trans "Edit pricing" %}</a>
<a href="{% url 'lingo-manager-pricing-matrix-slug-edit' object.pk matrix.criteria.slug %}">{% trans "Edit pricing" context 'matrix' %}</a>
pmarillonnet commented 2024-04-22 15:16:07 +02:00
Owner
Copy Link

Pas compris ici pourquoi il y a besoin d’un contexte i.e. de quelles autres façons ce “Edit pricing” pourrait se traduire ?

Pas compris ici pourquoi il y a besoin d’un contexte i.e. de quelles autres façons ce “Edit pricing” pourrait se traduire ?
lguerin commented 2024-04-22 15:42:31 +02:00
Author
Owner
Copy Link

On pourrait traduire par "modification de la grille tarifaire" ou "modification du tarif", ou encore "modification des tarifs".
Ca dépend du contexte, en somme :)

On pourrait traduire par "modification de la grille tarifaire" ou "modification du tarif", ou encore "modification des tarifs". Ca dépend du contexte, en somme :)
pmarillonnet commented 2024-04-23 08:59:56 +02:00
Owner
Copy Link

Ok j’avais clairement pas saisi cette nuance dans les traductions, très bien.

Ok j’avais clairement pas saisi cette nuance dans les traductions, très bien.
pmarillonnet marked this conversation as resolved
lguerin added 2 commits 2024-04-22 15:36:33 +02:00
lguerin added 1 commit 2024-04-22 15:37:32 +02:00
lguerin added 1 commit 2024-04-22 15:38:22 +02:00
pmarillonnet commented 2024-04-23 09:00:26 +02:00
Owner
Copy Link

(Je vais refaire un tour à tête reposée mais a priori on n’est pas loin du ack.)

(Je vais refaire un tour à tête reposée mais a priori on n’est pas loin du ack.)
pmarillonnet approved these changes 2024-04-23 15:03:11 +02:00
pmarillonnet left a comment
Owner
Copy Link

Ok j’ai refait un tour plus diagonalisant, rien à redire, c’est bon pour moi.

Ok j’ai refait un tour plus diagonalisant, rien à redire, c’est bon pour moi.
lguerin force-pushed wip/89605-pricing-reduction-min-price from 688d5157c2 to 6448b16220 2024-04-25 14:41:15 +02:00 Compare
lguerin merged commit 6448b16220 into main 2024-04-26 10:46:04 +02:00
lguerin deleted branch wip/89605-pricing-reduction-min-price 2024-04-26 10:46:04 +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#186
No description provided.
Delete Branch "wip/89605-pricing-reduction-min-price"

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?