inclusion-numerique / mon-suivi-social

GNU Affero General Public License v3.0
5 stars 0 forks source link

fix: sorting descending #41

Closed mlbiche closed 1 year ago

mlbiche commented 1 year ago

Tout petit correctif, mais ça m'intéresse de savoir ce que tu (@hugues-m) en penses ?

Le problème arrivait lorsqu'on cliquait une seconde fois sur le lien de tri. Lorsque le query param de l'ordre du tri était retiré de l'URL pour basculer sur descendant, le tri par défaut considéré était ascendant donc ça rentrait en conflit.

Je vois pas mal de TODO sur cette partie. C'est en chantier ?

Est-ce que ce serait une bonne idée d'uniformiser la présence/non-présence du query param de l'ordre au sein de l'application ? Parce que par exemple pour les bénéficiaires c'est l'inverse (absence = ascendant ou order=desc) ?

hugues-m commented 1 year ago

C'est un side effect auquel je n'avais pas pensé ! C'était encore un peu en chantier oui ...

Il y a un "default sorting" en l'absence de paramètres, qu'il faut a mon avis laisser, mais tu as raison que l'absence de sorting alors qu'on tri sur une autre propriété que celle par défaut ne devrait pas être à l'envers.

A voir ce qu'on peut extraire de robuste