Slashs manuels en fin d'URL #117

Open
opened 7 months ago by Lephenixnoir · 4 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 ?
Collaborator

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.
Poster
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...
Poster
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.
Poster
Owner

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

En attendant, le commit 277ec535e7f38ab3d50e9b88a4ff961cccac0b01 de @Eragon le fait manuellement.
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: devs/PCv5#117
Loading…
There is no content yet.