Development #12932
don't use same url prefix for connected and unconnected pages
0%
Description
We do base the theming on the URL prefix, e.g. we can apply a different template on https://.../accounts/ and it will apply to /accounts/, /accounts/change-email/, etc. We would use that /accounts/ prefix to include the dashboard links in the sidebar however the /accounts/ prefix is also used for unconnected page, such as /accounts/register or /accounts/password/reset.
It would therefore be quite useful if we had different prefixes there.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 5 ans
Travail d'inventaire à faire, sur les combo, chercher page.redirect like '%accounts%'.
De là déterminer l'ampleur du travail de migration.
S'il apparait raisonnable, on pourra se passer de phase de transition où authentic pourrait gérer deux formats d'URL (ce qui me semble un peu galère).
À déterminer, les URL qu'on souhaite :
- /accounts/* pour les manipulations d'utilisateur connecté (mes infos, éditer mon compte, changer mon mot de passe, etc.)
- /register/ pour l'inscription
- ? pour "mot de passe oublié"
- d'autres ?
Mis à jour par Benjamin Dauvergne il y a environ 2 ans
- Fichier 0001-urls-move-public-view-into-their-own-prefix-12932.patch 0001-urls-move-public-view-into-their-own-prefix-12932.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a environ 2 ans
- Statut changé de Solution proposée à En cours
Manque la gestion du cas avec paramètre token.
Mis à jour par Benjamin Dauvergne il y a environ 2 ans
- Statut changé de En cours à Solution validée
Et donc non ça marche tout seul en fait1.
1 https://github.com/django/django/blob/2.2.26/django/views/generic/base.py#L177
Mis à jour par Benjamin Dauvergne il y a environ 2 ans
- Fichier 0001-urls-move-public-view-into-their-own-prefix-12932.patch 0001-urls-move-public-view-into-their-own-prefix-12932.patch ajouté
- Statut changé de Solution validée à Solution proposée
+1 test pour le principe.
Mis à jour par Valentin Deniaud il y a environ 2 ans
Je valide le code mais je n'arrive pas à me convaincre de la pertinence de tout mettre sous /self/, je veux bien que quelqu'un d'autre valide ça.
Aussi il faudrait créer un ticket dans hobo pour modifier la variable idp_account_url.
Mis à jour par Benjamin Dauvergne il y a environ 2 ans
Si c'est juste le nommage ça ne me dérange pas de changer self pour autre chose, mais si c'est l'idée même de mettre ça sous une autre hiérarchie, c'est pour faciliter la gestion du thème, mais peut-être que le thème à la racine suffit et donc dans ce cas je met tout à la racine.
Mis à jour par Benjamin Dauvergne il y a environ 2 ans
- Fichier 0001-urls-move-public-view-into-their-own-prefix-12932.patch 0001-urls-move-public-view-into-their-own-prefix-12932.patch ajouté
Voilà, avec tout à la racine.
Mis à jour par Paul Marillonnet il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Ok.
Mis à jour par Benjamin Dauvergne il y a presque 2 ans
- Statut changé de Solution validée à Nouveau
- Assigné à
Benjamin Dauvergnesupprimé
Un peu frileux pour pousser finalement, je n'ai toujours pas bien compris si tout était ok coté combo ou si un travail supplémentaire était nécessaire.
Mis à jour par Benjamin Dauvergne il y a presque 2 ans
- Statut changé de Nouveau à Solution validée
- Assigné à mis à Benjamin Dauvergne
Mis à jour par Frédéric Péters il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 84cf3adbdb6a1e13306900c5a4b57786e63873e4 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Wed Jan 12 17:48:51 2022 +0100 urls: move public view into their own prefix (#12932)
Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
urls: move public view into their own prefix (#12932)