Projet

Général

Profil

Development #12932

don't use same url prefix for connected and unconnected pages

Ajouté par Frédéric Péters il y a plus de 7 ans. Mis à jour il y a presque 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
25 août 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

Révision 84cf3adb (diff)
Ajouté par Benjamin Dauvergne il y a presque 2 ans

urls: move public view into their own prefix (#12932)

Historique

#1

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 ?
#4

Mis à jour par Benjamin Dauvergne il y a presque 5 ans

  • Assigné à mis à Benjamin Dauvergne
#5

Mis à jour par Stéphane Laget il y a plus de 4 ans

ce qui serait utile à Chambéry : #37784

#6

Mis à jour par Benjamin Dauvergne il y a environ 2 ans

#7

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.

#8

Mis à jour par Benjamin Dauvergne il y a environ 2 ans

  • Statut changé de En cours à Solution validée
#9

Mis à jour par Benjamin Dauvergne il y a environ 2 ans

+1 test pour le principe.

#10

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.

#11

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.

#19

Mis à jour par Paul Marillonnet il y a presque 2 ans

  • Statut changé de Solution proposée à Solution validée

Ok.

#20

Mis à jour par Benjamin Dauvergne il y a presque 2 ans

  • Statut changé de Solution validée à Nouveau
  • Assigné à Benjamin Dauvergne supprimé

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.

#21

Mis à jour par Benjamin Dauvergne il y a presque 2 ans

  • Statut changé de Nouveau à Solution validée
  • Assigné à mis à Benjamin Dauvergne
#25

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)
#28

Mis à jour par Transition automatique il y a presque 2 ans

  • Statut changé de Résolu (à déployer) à Solution déployée
#29

Mis à jour par Transition automatique il y a presque 2 ans

Automatic expiration

Formats disponibles : Atom PDF