Projet

Général

Profil

Development #43080

construction fil d'ariane admin/{forms,workflows,categories}

Ajouté par Frédéric Péters il y a presque 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
19 mai 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Le bout d'insertion dans le fil d'ariane est répété à différents endroits alors qu'il pourrait juste être défini dans _q_traverse.


Fichiers

Révisions associées

Révision 19efdd29 (diff)
Ajouté par Frédéric Péters il y a presque 4 ans

admin: leave categories breadcrumb to wcs (#43080)

Révision 82ea582a (diff)
Ajouté par Frédéric Péters il y a presque 4 ans

admin: set forms/workflows breadcrumb in _q_traverse (#43080)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 4 ans

#2

Mis à jour par Nicolas Roche il y a presque 4 ans

J'ai trouvé 2 régressions : (URL de la page => URL du fil d'ariane)

Il manque un slash dans les URL du fil d'Ariane pour les workflows : Il y a répétition de 'Formulaire' dans le fils d’Ariane des catégories
  • https://wcs.dev.publik.love/backoffice/forms/categories/ => https://wcs.dev.publik.love/backoffice/forms/forms/
    wcs/admin/categories.py
    @@ -170,5 +170,4 @@ class CategoriesDirectory(Directory):
    
         def _q_index(self):
    -        get_response().breadcrumb.append( ('categories/', _('Categories')) )
             get_response().add_javascript(['jquery.js', 'jquery-ui.js', 'biglist.js',
                 'qommon.wysiwyg.js'])
    @@ -234,4 +233,3 @@ class CategoriesDirectory(Directory):
    
         def _q_traverse(self, path):
    -        get_response().breadcrumb.append(('forms/', _('Forms')))
             return super(CategoriesDirectory, self)._q_traverse(path)
    
#3

Mis à jour par Frédéric Péters il y a presque 4 ans

Merci, branche à jour pour inclure également nettoyage pour le fil d'ariane des catégories + patch attaché pour le module auquotidien pour aller avec.

#4

Mis à jour par Nicolas Roche il y a presque 4 ans

Désolé, il y avait une autre régression que je n'avais pas vue sur les modèles de fiches :
https://wcs.dev.publik.love/backoffice/cards/7/ => https://wcs.dev.publik.love/backoffice/forms/cards/

Elle celle là qui persiste sur les catégorie de formulaires :
https://wcs.dev.publik.love/backoffice/forms/categories/1/ => https://wcs.dev.publik.love/backoffice/forms/categories/categories/

(Pour auquotidien je fais confiance car je ne sais pas comment tester le module)

#5

Mis à jour par Frédéric Péters il y a presque 4 ans

il y avait une autre régression que je n'avais pas vue sur les modèles de fiches :

Vu, corrigée.

Elle celle là qui persiste sur les catégorie de formulaires :
https://wcs.dev.publik.love/backoffice/forms/categories/1/ => https://wcs.dev.publik.love/backoffice/forms/categories/categories/

Je ne comprends pas ce que tu essaies de pointer ici; sur quelle page il devrait y avoir quoi il y a quoi à la place ?

(Pour auquotidien je fais confiance car je ne sais pas comment tester le module)

Il fait partie du déploiement de publik-devinst, il n'y a rien de particulier. (et je me dis, que le point précédent est peut-être lié, et que tu utilises auquotidien sans y avoir appliqué le patch).

#6

Mis à jour par Nicolas Roche il y a presque 4 ans

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

et je me dis, que le point précédent est peut-être lié, et que tu utilises auquotidien sans y avoir appliqué le patch

tout juste

#7

Mis à jour par Frédéric Péters il y a presque 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 82ea582a0f54426e5625a87ef94d52ab86490a13
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue May 19 14:42:57 2020 +0200

    admin: set forms/workflows breadcrumb in _q_traverse (#43080)
#8

Mis à jour par Frédéric Péters il y a presque 4 ans

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

Formats disponibles : Atom PDF