Project

General

Profile

Development #43080

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

Added by Frédéric Péters 11 days ago. Updated 1 day ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
19 May 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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.

0001-admin-set-forms-workflows-breadcrumb-in-_q_traverse-.patch View (5.48 KB) Frédéric Péters, 19 May 2020 06:10 PM

0001-admin-leave-categories-breadcrumb-to-wcs-43080.patch View (1.18 KB) Frédéric Péters, 24 May 2020 07:16 PM

Associated revisions

Revision 19efdd29 (diff)
Added by Frédéric Péters 6 days ago

admin: leave categories breadcrumb to wcs (#43080)

Revision 82ea582a (diff)
Added by Frédéric Péters 1 day ago

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

History

#1 Updated by Frédéric Péters 11 days ago

#2 Updated by Nicolas Roche 6 days ago

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 Updated by Frédéric Péters 6 days ago

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 Updated by Nicolas Roche 5 days ago

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 Updated by Frédéric Péters 5 days ago

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 Updated by Nicolas Roche 5 days ago

  • Status changed from Solution proposée to 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 Updated by Frédéric Péters 1 day ago

  • Status changed from Solution validée to 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 Updated by Frédéric Péters 1 day ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF