Slashs manuels en fin d'URL #117

Closed
opened 2022-11-15 11:41:49 +01:00 by Lephenixnoir · 5 comments
Owner

Les URL pour les listings du forum nécessitent un / final, conformément à nos choix IIRC.

Cependant certaines URL explicites dans la navbar l'oublient, et sont donc 404 :

<a href='/forum/actus/calc'>Nouveautés Casio</a>
<a href='/forum/actus/projets'>Projets communutaires</a>
<a href='/forum/actus/evenements'>Événements de Planète Casio</a>
<a href='/forum/actus/autres'>Autres nouveautés</a>

On pourrait essayer de mettre une redirection mais je suis tenté de juste les mettre à la main et de se souvenir de la convention.

Des avis ?

Les URL pour les listings du forum [nécessitent un `/` final](https://gitea.planet-casio.com/devs/PCv5/src/commit/2b9ab64f6ef88300032b671b246297688ff4a359/app/routes/forum/index.py#L20), conformément à nos choix IIRC. Cependant certaines URL explicites dans la navbar [l'oublient](https://gitea.planet-casio.com/devs/PCv5/src/commit/2b9ab64f6ef88300032b671b246297688ff4a359/app/templates/base/navbar/news.html#L12-L15), et sont donc 404 : ```html <a href='/forum/actus/calc'>Nouveautés Casio</a> <a href='/forum/actus/projets'>Projets communutaires</a> <a href='/forum/actus/evenements'>Événements de Planète Casio</a> <a href='/forum/actus/autres'>Autres nouveautés</a> ``` On pourrait essayer de mettre une redirection mais je suis tenté de juste les mettre à la main et de se souvenir de la convention. Des avis ?
Member

La redirection est utile pour les personnes qui tappent l'url a la main, ou pour les copié-coller. Mais on devrait aussi fix les liens de la navbar pour qu'ils aillent directement au bon endroit.

La redirection est utile pour les personnes qui tappent l'url a la main, ou pour les copié-coller. Mais on devrait aussi fix les liens de la navbar pour qu'ils aillent directement au bon endroit.
Author
Owner

Right, en fait Flask est supposé rediriger mais je suppose que notre convertisseur bizarre l'en empêche. Je regarde si on peut faire quelque chose...

Right, en fait [Flask est supposé rediriger](https://flask.palletsprojects.com/en/2.2.x/quickstart/#unique-urls-redirection-behavior) mais je suppose que notre convertisseur bizarre l'en empêche. Je regarde si on peut faire quelque chose...
Author
Owner

Semble lié à #2533: path converter doesn't redirect to trailing slash.

Il y a une PR en attente de merge pour ce bug.

Semble lié à [#2533](https://github.com/pallets/werkzeug/issues/2533): *path converter doesn't redirect to trailing slash*. Il y a [une PR en attente de merge](https://github.com/pallets/werkzeug/pull/2535) pour ce bug.
Author
Owner

En attendant, le commit 277ec535e7 de @Eragon le fait manuellement.

En attendant, le commit 277ec535e7f38ab3d50e9b88a4ff961cccac0b01 de @Eragon le fait manuellement.
Owner

La PR a été merge, je suppose que c’est ok maintenant

La PR a été merge, je suppose que c’est ok maintenant
Darks closed this issue 2023-06-20 19:47:59 +02:00
Sign in to join this conversation.
No description provided.