search

FreederTeam / Freeder

An easy-to-use free feed reader
Other
5 stars 1 forks source link

message d'erreur d'import OPML #16

Closed tmos closed 10 years ago

tmos commented 10 years ago

Faudrait rendre ça un peu plus gentil, et l'afficher dors du retour à la page settings je pense. Some of the imported feeds encountered errors during refresh. The following feeds were NOT imported:array(11) { [0]=> string(33) "house-under-a-rock.tumblr.com/rss" [1]=> string(25) "http://anakrose.com/feed/" [2]=> string(28) "http://cabusar.fr/?feed=rss2" [3]=> string(28) "https://tiger-222.fr/rss.php" [4]=> string(37) "https://tiger-222.fr/shaarli//?do=rss" [5]=> string(25) "http://anakrose.com/feed/" [6]=> string(28) "http://cabusar.fr/?feed=rss2" [7]=> string(28) "https://tiger-222.fr/rss.php" [8]=> string(37) "https://tiger-222.fr/shaarli//?do=rss" [9]=> string(29) "https://lejournal.cnrs.fr/rss" [10]=> string(22) "http://tontof.net/?rss" }

Phyks commented 10 years ago

Ouais, la plupart des messages d'erreur sont moches et brut de décoffrage. Faut voir comment c'est le plus simple à gérer dans le template.

Tu as une idée de ce qu'il faudrait que je renvoie pour afficher un joli message d'erreur dans le template ?

tmos commented 10 years ago

J'aimerais afficher tous les messages d'erreur de façon standardisée. Typiquement dans ce genre (désolé pour la source de l'exemple) : modal_boxes_5

Je suis entrain de bosser sur ladite boite modale en ce moment, je vous tiens au courant quand j'ai trouvé une solution efficace (j'en ai besoin pour l'ajout des flux aussi).

eliemichel commented 10 years ago

Bah on peut balancer une variable $error avec un titre et un message, mais faut voir comment faire si la réaction à l'erreur dépend de celle-ci. Sinon, l'idée de boîte de dialogue me semble bonne. =)

tmos commented 10 years ago

En l'occurrence je pensais surtout a des erreurs de type "notification": l'utilisateur répond OK et modifie ensuite son opération. C'est ce que tu voulais dire ?

On 25 juillet 2014 21:14:34 CEST, Elie Michel notifications@github.com wrote:

Bah on peut balancer une variable $error avec un titre et un message, mais faut voir comment faire si la réaction à l'erreur dépend de celle-ci.

Reply to this email directly or view it on GitHub: https://github.com/FreederTeam/Freeder/issues/16#issuecomment-50192365

eliemichel commented 10 years ago

Oui dans ce cas y a pas de problème.

Phyks commented 10 years ago

Sympa la boîte de dialogue ! Je propose de :

Je m'occupe de ça pour mettre au bon format toutes les erreurs actuellement présentes dans le code.

Il va falloir réfléchir au formatage des erreurs, pour que ce soit facilement personnalisable par le thème. Si ce n'est pas le cas, me le signaler.

tmos commented 10 years ago

Ça me semble très bien comme intégration des messages systèmes. D'ailleurs, ça peut être des erreurs, mais aussi de simples avertissements non ? Ça pourrait être sympa d'avoir ce genre de catégorisation de messages :

Il faudrait donc transmettre au thème $error[type] $error[title] et $error[content].

On pourrait même penser à permettre de régler la verbosité de freeder, afin de ne pas avoir de messages inutiles pour les usagers non-admins (dans l'optique d'un multi users), ou pour ne pas s'encombrer de messages non bloquants si on à un usage un peu particulier de freeder.

Ça vous semble utile ? Vous en pensez quoi ?

Phyks commented 10 years ago

:+1: je fais ça

tmos commented 10 years ago

\o/ Comme ça en mettant le type d'erreur en classe CSS, on pourra donner un air plus ou moins méchant aux fenêtres de messages :D

eliemichel commented 10 years ago

Je serai favorable à l'idée d'ajouter une case « Ne plus afficher ce message », en particulier pour dire que tel flux est inactif. Comme ça si l'utilisateur veut pas un truc verbeux, Freeder se tait petit à petit. Du coup faudra une page de réglages pour les « Notifications ».

edit: je propose comme nom pour les types respectivement info, warning et error. Ça me semble assez canonique. Et on peut ajouter none pour la forme.

Phyks commented 10 years ago

Implémenté. Chaque erreur est transmise au thème dans une variable $error.

À disposition: