manager: change day/week/month urls (#72323) #20
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/72323-day-week-month-links"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
(finalement le js moche de https://gitea.entrouvert.org/entrouvert/chrono/pulls/19 m'a bien aidée pour le date-picker week et les nouvelles urls)
d9fbc2b08e
toba6a42b75a
On table sur le fait que personne ne bookmark l'URL d'un mois donc on ne met pas de redirection ?
(moi qui utilise mon historique comme un gestionnaire de bookmark je vais me prendre des 404 tout le temps mais si c'est que moi ça va)
ba6a42b75a
to5baba1f8e4
Pas réfléchi à ça. Tu penses qu'on devrait ? Mais alors pour toutes les anciennes urls day/week/month ?
De mon côté je me suis posé la question en rédigeant le ticket et je n'ai rien mentionné parce que je n'avais pas d'opinion arrêtée. Presque sûr personne ne va avoir de bookmark "juin 2022", par exemple.
En zone intermédiaire je serais prêt à dire qu'on peut faire le minimum pour éviter la 404 et sur les anciennes formes rediriger vers la racine de l'agenda.
Ça m'irait bien, mon cas d'usage c'est effectivement ça : je veux aller voir l'agenda machin-bidule sur l'instance truc-muche, je tape « machin-bidule agenda truc-muche » dans la barre de mon navigateur, il me trouve un lien truc-muche.eo.org/manage/blabla/2021/01/, je vais dessus et je navigue jusqu'à la page qui m'intéresse. Si je me prends une 404 je dois aller bidouiller l'url, mais si je suis redirigé sur une page naviguable je suis content.
Ça me semble aussi couvrir le cas d'un poste laissé en veille le soir, le matin après mise à en prod il va y avoir un clic qui va avoir un comportement inattendu, redirection vers la racine au lieu de la date ciblée, mais c'est vite oublié, davantage qu'une 404 en tout cas.
2e commit avec des redirections
C'est ok mais il y a un rebase manuel à faire, en partie à cause de #72324 (boutons de navigation) en partie en cause de #72321 (th weeknum); j'ai rapidement regardé ça semble assez basique.