Development #36578
affichage des warnings
0%
Description
Les cubes contiennent maintenant un clé warnings contenant une liste de messages, il faudrait les afficher.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Lié à Development #36576: ajouter le champ "warnings" au schéma des cubes ajouté
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Fichier 0001-show-cube-warnings-36576.patch 0001-show-cube-warnings-36576.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
- de grandes chances que personne ne les voit,
- mais c'est simple.
- au moins une fois les afficher en pleine figure de l'utilisateur, parce que ça peut donner immédiatement des indications sur ce qui ne va pas,
- pouvoir en oublier certains : certains champs ne serviront jamais aux statistiques et n'ont donc pas de nom de variable définis (genre un booléen qui permet juste d'afficher une section du formulaire) mais le warning sera toujours là.
Mis à jour par Serghei Mihai il y a plus de 4 ans
Il faut sortir le bloc des messages du form et aussi rajouter une classe genre "cube-warnings" pour pouvoir derrière styler et rendre plus visible à l'usager.
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Statut changé de Solution proposée à En cours
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
Le souci c'est que c'est cube-form qui constitue la colonne de droite, faut que je crée un div pour construire la colonne désormais si je sorts les warnings de là pour que tout ça reste groupé.
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Fichier 0001-show-cube-warnings-36576.patch 0001-show-cube-warnings-36576.patch ajouté
- Statut changé de En cours à Solution proposée
Je suis trop incompétent pour revoir le layout sur ce ticket, je préfère me limiter à ajouter une classe, si quelqu'un veut refaire les templates pour se rapprocher des standard ailleurs que ce soit fait dans un ticket dédié. J'ai quand même ajouté une clase cube-warnings (mais je ne suis pas certain du besoin là maintenant justement vu qu'il y a pas de style pour en faire quelque chose).
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Lié à Development #30866: organisation page (titre/actions/contenu) ajouté
Mis à jour par Nicolas Roche il y a plus de 4 ans
- Statut changé de Solution proposée à Solution validée
J'ai l'impression que la proposition de Serghei serait que si tu déplaces tes ajouts dans le gabarit sous </form>, alors les warnings seront affichés en haut (au dessus du titre) comme d'habitude.
(j'ai testé en bidouillant mes cubes dans /var/lib/bijoe/tenants/bijoe.dev.publik.love/schemas/wcs_dev_publik_love.mode)
Le temps de pouvoir oublier certains warnings, je comprend aussi si tu préfères les placer à l'endroit où on les voit les moins, même si je suis plutôt de l'avis de Serghei.
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
Nicolas Roche a écrit :
J'ai l'impression que la proposition de Serghei serait que si tu déplaces tes ajouts dans le gabarit sous
</form>, alors les warnings seront affichés en haut (au dessus du titre) comme d'habitude.
(j'ai testé en bidouillant mes cubes dans /var/lib/bijoe/tenants/bijoe.dev.publik.love/schemas/wcs_dev_publik_love.mode)
Je veux bien un screenshot voir ce que ça donne.
Le temps de pouvoir oublier certains warnings, je comprend aussi si tu préfères les placer à l'endroit où on les voit les moins, même si je suis plutôt de l'avis de Serghei.
Je n'avais pas compris cela du tout ; oui si ça va à tout le monde je peux les mettre là, comme je le disais plus haut je n'étais pas très inspiré, ma priorité vu que c'est quand même rare d'avoir des warnings c'était qu'on les affiche, déjà.
Mis à jour par Nicolas Roche il y a plus de 4 ans
- Fichier Screenshot_2020-01-23 Statistiques.png ajouté
Mis à jour par Nicolas Roche il y a plus de 4 ans
- Fichier
Screenshot_2020-01-23 Statistiques.pngsupprimé
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Fichier 0001-show-cube-warnings-36576.patch 0001-show-cube-warnings-36576.patch ajouté
- Statut changé de Solution validée à Solution proposée
Ok.
Mis à jour par Nicolas Roche il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 5d1524702abe166fd57b12d1fd01dfc061745907
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Tue Oct 1 13:33:33 2019 +0200
show cube warnings (#36578)Mis à jour par Valentin Deniaud il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
show cube warnings (#36578)