manager: change day/week/month urls (#72323) #20

Merged
lguerin merged 2 commits from wip/72323-day-week-month-links into main 2022-12-26 18:17:55 +01:00
Owner

(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)

(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)
lguerin force-pushed wip/72323-day-week-month-links from d9fbc2b08e to ba6a42b75a 2022-12-13 17:55:24 +01:00 Compare
Owner

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)

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)
lguerin force-pushed wip/72323-day-week-month-links from ba6a42b75a to 5baba1f8e4 2022-12-13 18:05:34 +01:00 Compare
Author
Owner

Pas réfléchi à ça. Tu penses qu'on devrait ? Mais alors pour toutes les anciennes urls day/week/month ?

Pas réfléchi à ça. Tu penses qu'on devrait ? Mais alors pour toutes les anciennes urls day/week/month ?
Owner

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.

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

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.

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

2e commit avec des redirections

2e commit avec des redirections
lguerin changed target branch from wip/72322-week-monday-date to main 2022-12-26 09:06:48 +01:00
fpeters approved these changes 2022-12-26 17:37:02 +01:00
fpeters left a comment
Owner

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.

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.
lguerin added 2 commits 2022-12-26 18:10:44 +01:00
lguerin merged commit b0201494cc into main 2022-12-26 18:17:55 +01:00
lguerin deleted branch wip/72323-day-week-month-links 2022-12-26 18:17:55 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/chrono#20
No description provided.