Facturation: ajouter des liens vers les objets depuis la page des lignes non facturées (#75100) #34
No reviewers
pmarillonnet
Labels
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: entrouvert/lingo#34
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/75100-invoicing-non-invoiced-lines-links"
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?
0001: pour harmoniser les écrans
0002: le dev
0003: déplacement de tests qui dépendent de la régie seulement et non d'une campagne
cf64dd80e7to591bd0afaa591bd0afaato9e6c2905c69e6c2905c6tocb4edcb6a0cb4edcb6a0to933fe292da933fe292datod01faf0c05Merci pour le rebasage, je commence à relire.
Quelques petites remarques à la lecture du code, plus pour ma compréhension personnelle qu’autre chose.
@ -482,0 +485,4 @@if not settings.KNOWN_SERVICES.get('chrono'):returnchrono = list(settings.KNOWN_SERVICES['chrono'].values())[0]chrono_url = chrono.get('url') or ''Pas compris pourquoi on cherche quand même à retrouver l’URL de l’événement chrono lorsque
chrono.get('url')est évalué àFalseet qu’on se retrouve avec une chaîne vide à la place. C’est quoi le cas qui justifierait cela et dans lequel on ne se retrouverait pas avec une URL erronée au final ?En effet, ça doit etre un copier/coller d'ailleurs, je vais changer ça
@ -48,2 +48,3 @@</td><td><a class="details-toggle" href="#">{% trans "see details" %}</a></td><td><span class="togglable"></span>Ok, pas très à l’aise avec le bout de JS qui vient remplir ces
<span class="toggable"/>d’abord vides dans les gabarits, alors je vais tenter de recopier les données du test écrit dans un shell django, et constater le résultat.Ok, testé en local, c’est bon pour moi.
@ -36,0 +36,4 @@{% if line.event.agenda %}<br /><a href="{% url 'lingo-manager-agenda-detail-redirect' line.event.agenda %}">{% trans "see agenda" %}</a>{% if line.chrono_event_url %}- <a href="{{ line.chrono_event_url }}">{% trans "see event" %}</a>{% endif %}Juste par curiosité, c’est dans quel cas qu’on est en mesure de rediriger vers
lingo-manager-agenda-detailmais pas en mesure de retrouver une URL d’événement correspondant côté chrono ?C’est pour tâcher d’avoir un fonctionnement un minimum standalone dans lingo, ou bien il y a vraiment des cas où ça survient ?
Non en effet ça ne devrait pas arriver, si on a pu lancer une campagne de facturation, c'est qu'on a pu synchroniser les agendas chronos; aussi, dans les lignes de facturation, on doit retrouver dans le json
eventle slug de l'event, et on doit être en mesure d'avoir une url.Ce ne sont que des précautions :)
Ok, c’est ce qu’il me semblait, merci.
@ -64,0 +62,4 @@var $tr = $toggle.parents('tr');var line_id = $tr.data('line-id');var $details = $('tr[data-details-for-line-id=' + line_id + ']');$tr.toggleClass('toggled').toggleClass('untoggled');Typiquement le genre de ligne où je comprends rien :)
Même en lisant la doc JQuery c’est pas plus clair.
Je vais me contenter d’injecter des données dans mon devinst et tâcher de reproduire le comportement de repliabilité/dépliabilité, ça vaudra pour relecture du code…
@ -200,0 +462,4 @@'agenda': 'foobar',}line.save()resp = app.get('/manage/invoicing/regie/%s/non-invoiced-lines/' % regie.pk)Ok, le test est clair. Nickel.
Juste comme dit plus haut, j’ai du mal à voir ”dans la vraie vie” quand surviendraient les cas où le lien “see agenda” est visible, mais où il manque le nom d’agenda et le slug d’événement nécessaires à la génération du lien correspondant coté chrono.
(Oops, je voulais juste publier les commentaires et en fait j’ai approuvé la PR, pas bien grave, les commentaires ne remettent pas vraiment en question le code écrit.)
Ack pour de vrai cette fois-ci.