Project

General

Profile

Bug #71157

wcs: dans la cellule "Demandes de l'usager" les demandes devraient toutes être triées anti-chronologiquement et pas seulement pour chaque site

Added by Benjamin Dauvergne 3 months ago. Updated 3 months ago.

Status:
Nouveau
Priority:
Bas
Assignee:
-
Target version:
-
Start date:
08 November 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

Si des demandes de plusieurs sites sont affichées, on ne voit pas forcément les demandes récentes des sites suivants qui sont cachées par les demandes moins récentes des sites précédents (surtout si l'utilisateur a faussement l'impression que tout est bien trié anti-chronologiquement).

Cf. ticket client #71109

PS:
Les templates actuels font en gros :

{% for slug, forms in current_forms.items %}
      <div class="... user/current-forms-{{ slug }} ...">
      {% for data in forms.data|dictsortreversed:"form_receipt_datetime" %}
      ...
      {% endfor %}
      </div>
{% endfor %}

Il semble que le principal intérêt de ce découpage et de pouvoir appliquer cette classe "user/current-forms-{{ slug }}" aux sous liste de demandes, ce qui ne semble utilisé nul part dans publik-base-theme (j'ai fait un bête grep avec les deux classes et rien ne ressort).

History

#2

Updated by Benjamin Dauvergne 3 months ago

  • Description updated (diff)
#3

Updated by Benjamin Dauvergne 3 months ago

  • Priority changed from Normal to Bas

Et donc je vois que le cas s'est déjà présenté et est géré via une autre variable de contexte, forms :

templates/variants/montpellier/combo/wcs/current_forms.html:        {% for data in forms.data|dictsortreversed:"form_receipt_date" %}
templates/variants/strasbourg-2018/combo/wcs/current_forms.html:    {% for form in forms|dictsortreversed:"form_receipt_datetime" %}
templates/variants/strasbourg-2018/combo/wcs/user_all_forms.html:    {% for form in forms|dictsortreversed:"form_receipt_datetime"|slice:":20" %}
templates/variants/toodego/portal-user/combo/wcs/user_all_forms.html:    {% for form in forms|dictsortreversed:"last_update_time" %}

Je vais mettre en priorité bas et puis faire pareil dans le thème Toulouse pour l'instant.

#4

Updated by Benjamin Dauvergne 3 months ago

Je me dis quand même que le cas normal c'est celui où on veut tout trier par last_update_datetime (je ne vois pas d'intérêt à la façon actuelle de faire).

#5

Updated by Benjamin Dauvergne 3 months ago

Benjamin Dauvergne a écrit :

Je me dis quand même que le cas normal c'est celui où on veut tout trier par last_update_datetime (je ne vois pas d'intérêt à la façon actuelle de faire).

J'affine :
  • pour current_forms -last_update_time
  • pour done_forms -receipt_time parce qu'on ne veut pas les voir bouger une fois qu'elles sont terminées
  • pour all_forms -last_update_time pour les demandes en cours et -receipt_time pour les demandes terminées

Ça nécessiterait de trier dans get_context_data().

#6

Updated by Paul Marillonnet 3 months ago

Le truc c’est qu’il y a pas mal d’existant bâti autour de ce regroupement par site. Dans les thèmes je vois notamment :

~/src/publik-base-theme$ git grep list-of-forms | grep -v drafts
static/boussu-2022/_custom.scss:381:div.links-list.list-of-forms ul > li > a {
static/braine-l-alleud/_custom.scss:350:div.links-list.list-of-forms ul > li > a {
static/cap-atlantique/_custom.scss:126:.list-of-forms-cell {
static/cap-atlantique/_custom.scss:156:    @extend .list-of-forms-cell;
static/clermont-ici/_custom.scss:283:.list-of-forms-cell {
static/clermont-ici/_custom.scss:456:        @extend .list-of-forms-cell;
static/cnil/_custom.scss:249:div.list-of-forms {
static/grandlyon/_custom.scss:63:div.list-of-forms span.form-number {
static/haguenau/_custom.scss:370:.list-of-forms-cell {
static/haguenau/_custom.scss:443:    @extend .list-of-forms-cell;
static/includes/_cells.scss:303:div.list-of-forms {
static/isere-cd38/_custom.scss:440:    .list-of-forms {
static/ma-cnil/_custom.scss:249:div.list-of-forms {
static/meudon-2021/_custom.scss:330:.list-of-forms-cell {
static/meudon-2021/_custom.scss:447:    @extend .list-of-forms-cell;
static/meuse-cd55/_custom.scss:154:div.list-of-forms span.form-number {
static/montreuil/_custom.scss:208:div.list-of-forms {
static/rouen/_custom.scss:660:div.list-of-forms span.form-number {
static/toulouse-2022/_custom.scss:337:div.list-of-forms span.form-status {
static/toulouse-metropole/_custom.scss:1746:div.list-of-forms span.form-number {
static/toulouse/_custom.scss:1781:div.list-of-forms span.form-number {
templates/variants/grenoble-metropole-2019/combo/wcs/current_forms.html:4:    <div class="current-forms-{{ slug }} current-forms list-of-forms">
templates/variants/grenoble-metropole-2019/combo/wcs/user_done_forms.html:4:    <div class="user-forms-{{ slug }} list-of-forms">
templates/variants/rouen/combo/wcs/current_forms.html:14:      <div class="links-list current-forms-{{ slug }} current-forms list-of-forms">

Si on change ça il faut repasser partout dans ces thèmes :/

Also available in: Atom PDF