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
0%
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).
Historique
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Priorité changé de Normal à 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.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
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).
Mis à jour par Benjamin Dauvergne il y a plus d'un an
Benjamin Dauvergne a écrit :
J'affine :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).
- 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().
Mis à jour par Paul Marillonnet il y a plus d'un an
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 :/