Projet

Général

Profil

Bug #6177

ajouter la notion /tryauth (et /auth) sur la page d'accueil et sur les catégories

Ajouté par Thomas Noël il y a plus de 9 ans. Mis à jour il y a plus de 8 ans.

Statut:
Fermé
Priorité:
Haut
Assigné à:
Version cible:
-
Début:
17 décembre 2014
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

On a ajouté un /tryauth et /auth sur les formulaires (#5625) ; il serait bénéfique d'avoir aussi ces mécanismes sur :
  • la page d'accueil de wcs
  • chaque catégorie

Fichiers

Révisions associées

Révision 2d2e422e (diff)
Ajouté par Serghei Mihai il y a environ 9 ans

categories and homepage: add /auth and /tryauth paths (#6177)

Révision 877a98d3 (diff)
Ajouté par Serghei Mihai il y a environ 9 ans

homepage: add '/auth' and '/tryforms' path (#6177)

Historique

#1

Mis à jour par Frédéric Péters il y a environ 9 ans

  • Priorité changé de Normal à Haut
#2

Mis à jour par Frédéric Péters il y a environ 9 ans

  • Assigné à Frédéric Péters supprimé
#3

Mis à jour par Serghei Mihai il y a environ 9 ans

Patchs pour wcs et auquo

#4

Mis à jour par Frédéric Péters il y a environ 9 ans

Indentation ratée, ça ne devrait pas passer un "git show".

Il y a déjà une transmission automatique de root.py vers forms/root.py, ça m'irait bien d'avoir auth/tryauth uniquement dans ce second; ce qui permettra par la même occasion de gérer get_root_url() (c'est-à-dire les moments où la racine du wcs n'est pas /) pour les catégories.

#5

Mis à jour par Serghei Mihai il y a environ 9 ans

  • Fichier 0001-homepage-and-categories-add-auth-and-tryauth-paths-6.patch ajouté

Quelque chose dans ce genre, si je comprend bien

#6

Mis à jour par Frédéric Péters il y a environ 9 ans

Les catégories ont déjà une méthode get_url(), inutile d'en reconstituer une ici. [suppression d'un passage sur la config idp ou pas]. À regarder davantange, les tryauth et auth existants pourraient être partagés, pour en arriver à deux fonctions auth/tryauth recevant un paramètre url, et ainsi ramener les méthodes à 1) déterminer une URL, 2) passer la main à la fonction correspondante.

#7

Mis à jour par Serghei Mihai il y a environ 9 ans

  • Fichier 0001-homepage-and-categories-add-auth-and-tryauth-paths-6.patch supprimé
#8

Mis à jour par Serghei Mihai il y a environ 9 ans

  • Fichier 0001-homepage-and-categories-add-auth-and-tryauth-paths-6.patch supprimé
#9

Mis à jour par Serghei Mihai il y a environ 9 ans

  • Fichier 0001-homepage-and-categories-add-auth-and-tryauth-paths-6.patch ajouté

Nouveau patch tenant compte des dernieres remarques

#10

Mis à jour par Thomas Noël il y a environ 9 ans

Je dirais, au moins, revoir les commentaires.

#11

Mis à jour par Serghei Mihai il y a environ 9 ans

  • Fichier 0001-homepage-and-categories-add-auth-and-tryauth-paths-6.patch ajouté

Commentaires revus, petites améliorations du code

#12

Mis à jour par Thomas Noël il y a environ 9 ans

  • Fichier 0001-homepage-and-categories-add-auth-and-tryauth-paths-6.patch ajouté
  • Assigné à mis à Serghei Mihai

Plus on est de fous...

(ma version: commentaires tous déplacés, et tryauth déclarée avant auth, comme dans le code de départ)

#13

Mis à jour par Serghei Mihai il y a environ 9 ans

  • Fichier 0001-homepage-and-categories-add-auth-and-tryauth-paths-6.patch supprimé
#14

Mis à jour par Serghei Mihai il y a environ 9 ans

  • Fichier 0001-homepage-and-categories-add-auth-and-tryauth-paths-6.patch supprimé
#15

Mis à jour par Serghei Mihai il y a environ 9 ans

  • Fichier 0001-homepage-and-categories-add-auth-and-tryauth-paths-6.patch supprimé
#16

Mis à jour par Serghei Mihai il y a environ 9 ans

  • Fichier 0001-homepage-and-categories-add-auth-and-tryauth-paths-6.patch ajouté

Refait

#17

Mis à jour par Serghei Mihai il y a environ 9 ans

  • Fichier 0001-homepage-and-categories-add-auth-and-tryauth-paths-6.patch supprimé
#18

Mis à jour par Serghei Mihai il y a environ 9 ans

Je suis preneur d'idées ou placer les fonctions tryauth et auth car forms/root.py n'est certainement pas l'endroit approprié.

#19

Mis à jour par Frédéric Péters il y a environ 9 ans

Pour une des fonctions le "This URL" du commentaire a été retiré, et pas pour l'autre; je n'ai pas de problème à avoir ça dans forms/root.py.

#20

Mis à jour par Serghei Mihai il y a environ 9 ans

Ok, je corrige le commentaire et je push

#21

Mis à jour par Frédéric Péters il y a environ 9 ans

  • Statut changé de Nouveau à Résolu (à déployer)

Concierge, j'écris ton nom.

Frédéric.

commit 2d2e422e3045df42a06230062aaccac874af7b83
Author: Serghei Mihai <smihai@entrouvert.com>
Date:   Thu Apr 2 18:54:14 2015 +0200

    categories and homepage: add /auth and /tryauth paths (#6177)
#22

Mis à jour par Thomas Noël il y a plus de 8 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF