Development #7231
Interface de gestion publique du porte-document
0%
Description
Il faudrait un accès hors popup au porte-document; ça pourrait partir du home actuel, mais décoré.
Fichiers
Révisions associées
general: use gadjo theme for templates (#7231)
general: use gadjo theme for templates (#7231)
general: use separate view and template for document pick (#7231)
Historique
Mis à jour par Frédéric Péters il y a presque 9 ans
- Assigné à mis à Serghei Mihai (congés, retour 15/05)
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 9 ans
- Fichier 0001-use-gadjo-theme-for-templates-7231.patch 0001-use-gadjo-theme-for-templates-7231.patch ajouté
- Patch proposed changé de Non à Oui
Avec gadjo?
Mis à jour par Frédéric Péters il y a presque 9 ans
Ça peut être avec gadjo même si communément ça va plutôt prendre le thème public (via le partage du thème entre combo et les autres).
Sur le patch proposé, il faut faire attention à ce que le mode popup continue à fonctionner, ce qui ne m'a pas l'air le cas ici.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 9 ans
- Fichier 0001-use-gadjo-theme-for-templates-7231.patch 0001-use-gadjo-theme-for-templates-7231.patch ajouté
- Fichier 0001-base-template-for-fargo-app-7231.patch 0001-base-template-for-fargo-app-7231.patch ajouté
Proposition de template pour le thème publik ainsi que le pop-up d'upload sans le thème gadjo
Mis à jour par Frédéric Péters il y a presque 9 ans
Sur le patch proposé, il faut faire attention à ce que le mode popup continue à fonctionner, ce qui ne m'a pas l'air le cas ici.
Vraiment, quand je cliquerai sur "utiliser un fichier de mon porte-doc" dans wcs et qu'une popup fargo s'affichera (avec comme contenu https://fargo.imio.entrouvert.org/?pick=https%3A//wcs.imio.entrouvert.org/fargo/pick) ce contenu n'aura pas toute la décoration du thème ?
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 9 ans
- Fichier 0001-use-gadjo-theme-for-templates-7231.patch 0001-use-gadjo-theme-for-templates-7231.patch ajouté
- Fichier 0001-base-template-for-fargo-app-7231.patch 0001-base-template-for-fargo-app-7231.patch ajouté
J'ai séparé le template lors du "pick" de celui de la page d'accueil dans fargo en y mettant le stricte nécessaire pour le rendu du listing des fichiers.
Aussi un base.html pour le thème publik
Mis à jour par Frédéric Péters il y a presque 9 ans
def get_template_names(self):
"""
Returns a list of template names to be used for the request. Must return
a list. May not be called if render_to_response is overridden.
"""
Le patch retourne une chaine de caractères (et oui dans une version particulière de django ça peut marcher mais il n'y a vraiment pas de garantie de ce comportement).
Sur la structure générale, on a une vue qui fait deux choses (accueil + popup de sélection), avec un home.html qui inclut un fargo/files.html qui inclut un fargo/upload.html; j'ai l'impression que tout ça pourrait être simplifié en ne partageant pas la même URL pour la sélection et pour l'accueil de l'interface publique. Plutôt que jouer sur le get_template_names(), pour séparer le "pick", je préférerais avoir une vue juste pour lui.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 9 ans
- Fichier 0001-use-separate-view-and-template-for-document-pick.patch 0001-use-separate-view-and-template-for-document-pick.patch ajouté
- Fichier 0002-use-gadjo-theme-for-templates-7231.patch 0002-use-gadjo-theme-for-templates-7231.patch ajouté
Ok.
J'ai fait 2 patchs du coup.
Mis à jour par Frédéric Péters il y a presque 9 ans
- Statut changé de Nouveau à Résolu (à déployer)
Voilà, je viens de pousser les deux commits dans le dépôt (en mettant la référence du ticket dans le premier).
J'ai aussi poussé la modification suivante dans auquotidien :
commit c9c033c70376f6c3852f6aede9733db08967609b
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sun Jul 19 20:13:30 2015 +0200
fargo: adapt to URL change (#7231)
Mis à jour par Frédéric Péters il y a presque 9 ans
- Statut changé de Résolu (à déployer) à En cours
- Patch proposed changé de Oui à Non
Je retire ce que j'ai écrit et j'ai poussé une annulation des patchs, parce que j'avais vraiment mal lu et mal testé, et que ça ne va pas du tout :
- Depuis auquotidien, choisir de prendre un fichier depuis fargo, cliquer sur annuler, "MultiValueDictKeyError at /pick "'url'""
- Depuis auquotidien, choisir de prendre un fichier depuis fargo, décider d'uploader un fichier, "TypeError at /pick; super(type, obj): obj must be an instance or subtype of type"
Mis à jour par Frédéric Péters il y a presque 9 ans
- Statut changé de En cours à Résolu (à déployer)
J'ai finalement décidé de corriger les deux soucis et j'ai poussé ainsi.
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
- Statut changé de Résolu (à déployer) à Fermé
general: use separate view and template for document pick (#7231)