Project

General

Profile

Development #69546

intégrer la redirection /formulaire/ → /categorie/formulaire/

Added by Frédéric Péters over 1 year ago. Updated over 1 year ago.

Status:
Fermé
Priority:
Normal
Target version:
-
Start date:
25 September 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Ce code est actuellement dans au-quotidien (avec ses bugs), le taf devrait se faire dans w.c.s. (sans les bugs).


Files

Associated revisions

Revision efe6c53f (diff)
Added by Frédéric Péters over 1 year ago

misc: remove redirection to /category/formdef, handled by w.c.s. (#69546)

Revision 025d7e46 (diff)
Added by Frédéric Péters over 1 year ago

misc: redirect form URL to include category slug (#69546)

Revision 31c5db7b (diff)
Added by Frédéric Péters over 1 year ago

Revert "misc: remove redirection to /category/formdef, handled by w.c.s. (#69546)"

This reverts commit efe6c53faffafd77527caf89833e6139cae6963b.

Revision 3c9702e5 (diff)
Added by Frédéric Péters over 1 year ago

Revert "misc: redirect form URL to include category slug (#69546)"

This reverts commit 025d7e468d8f82bd8e01f0eb99161e501324642e.

Revision 8f43c01b (diff)
Added by Frédéric Péters over 1 year ago

misc: redirect form{def,data} URL to include category slug (#69546)

Revision 4ab604a8 (diff)
Added by Frédéric Péters over 1 year ago

misc: remove redirection to /category/formdef, handled by w.c.s. (back) (#69546)

History

#1

Updated by Frédéric Péters over 1 year ago

Ça amène quelques bouts moche mais ça doit gérer correctement les cas un peu nuls qu'on rencontrait type catégorie du nom d'une démarche.

(le comportement est plus uniforme mais ça fait quelque chose d'un peu différent d'auquotidien qui parfois affichait le formulaire sur /formdef/ sans redirection)

#2

Updated by Benjamin Dauvergne over 1 year ago

  • Status changed from Solution proposée to Solution validée
#3

Updated by Frédéric Péters over 1 year ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 025d7e468d8f82bd8e01f0eb99161e501324642e
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sun Oct 16 15:58:22 2022 +0200

    misc: redirect form URL to include category slug (#69546)
#4

Updated by Frédéric Péters over 1 year ago

  • Status changed from Résolu (à déployer) to En cours

Nope; ça oublie qu'il faut aussi les redirections pour /formulaire/<id>/; j'ai reverté :/

#5

Updated by Frédéric Péters over 1 year ago

Voilà.

Et pour être honnête par rapport à l'ancien code qui tentait de jouer via _q_traverse, il y a le cas de l'url de catégorie sans / final qu'on gère moins bien (mais perso ça me va).

(aussi, je me suis demandé ici s'il était opportun de modifier les retours de l'API, pour inclure la catégorie dans les URL, mais j'ai continué à laisser de côté).

#6

Updated by Thomas Noël over 1 year ago

  • Status changed from Solution proposée to Solution validée

Good.

Je me demandais pourquoi on ne voulait pas profiter de l'occasion pour supprimer cette redirection, mais le cas de la catérogie qui a le même slug qu'un formulaire suffit à ce que ça reste un bon fonctionnement. Cependant je reste d'accord que l'URL canonique, par exemple celle dans les API, reste sans le slug de catégorie ; ça autorise ainsi de modifier la catégorie d'un formulaire sans accroc (du moins dans une certaine mesure). Et j'arrête mon blabla.

#7

Updated by Frédéric Péters over 1 year ago

(je laisse de toute façon ça pour le cycle prochain, je ne vais pas le risquer à nouveau)

#8

Updated by Frédéric Péters over 1 year ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 8f43c01beabf4b859665203480926c12186d4470
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Oct 21 13:14:06 2022 +0200

    misc: redirect form{def,data} URL to include category slug (#69546)
#9

Updated by Transition automatique over 1 year ago

  • Status changed from Résolu (à déployer) to Solution déployée
#10

Updated by Transition automatique over 1 year ago

Automatic expiration

Also available in: Atom PDF