Development #13775
ajout manage/menu.json (en vue de faire apparaitre «Passerelle» dans le menu latéral Publik)
0%
Description
Aujourd'hui passerelle apparait comme entrée sous hobo («Services»). Mais on a des cas où la gestion de Passerelle peut être autorisée sans accès à hobo (par exemple, les instances où on veut permettre l'accès à la gestion des CSV, ou la vue des endpoints, etc).
Je serais donc pour ajouter un manage/menu.json dans Passerelle (et la partie correspondante dans hobo pour la construction du menu Publik complet)
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 7 ans
Côté Passerelle ce serait un revert de a8e7f2661622d24ce8c8e023459af05488176d53, en gros.
Mis à jour par Thomas Noël il y a plus de 7 ans
- Fichier 0001-Revert-general-remove-menu.json-handling-8590.patch 0001-Revert-general-remove-menu.json-handling-8590.patch ajouté
- Statut changé de Nouveau à Résolu (à déployer)
- Patch proposed changé de Non à Oui
Et donc, carrément, voici.
Mis à jour par Thomas Noël il y a plus de 7 ans
- Fichier 0001-re-add-menu.json-handling-13775.patch 0001-re-add-menu.json-handling-13775.patch ajouté
message de commit un peu moins couillon
Mis à jour par Frédéric Péters il y a plus de 7 ans
C'est du coverage assez facilement gagné en récupérant un test depuis combo genre; faut pas s'en priver.
Mis à jour par Thomas Noël il y a plus de 7 ans
- Fichier 0001-re-add-menu.json-handling-13775.patch 0001-re-add-menu.json-handling-13775.patch ajouté
Frédéric Péters a écrit :
C'est du coverage assez facilement gagné en récupérant un test depuis combo genre; faut pas s'en priver.
Surtout quand ça permet de voir que le code était du grand n'importe quoi.
Mis à jour par Frédéric Péters il y a plus de 7 ans
Je vois que ce n'est pas le cas dans les autres modules mais en jsonp il faudrait changer le content type pour être application/javascript
Mis à jour par Thomas Noël il y a plus de 7 ans
- Fichier 0001-re-add-menu.json-handling-13775.patch 0001-re-add-menu.json-handling-13775.patch ajouté
Et hop, avec un test pas bien rusé mais suffisant pour moi.
Mis à jour par Thomas Noël il y a plus de 7 ans
- Sujet changé de Faire apparaitre «Passerelle» dans le menu latéral Publik à ajout manage/menu.json (en vue de faire apparaitre «Passerelle» dans le menu latéral Publik)
commit 81c785d189a9b8fa005b0cffae57885a3bad8c98
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Thu Oct 27 17:16:55 2016 +0200
re-add menu.json handling (#13775)
This reverts commit a8e7f2661622d24ce8c8e023459af05488176d53 (#8590)
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
- Statut changé de Résolu (à déployer) à Fermé
re-add menu.json handling (#13775)
This reverts commit a8e7f2661622d24ce8c8e023459af05488176d53 (#8590)