Testez ZdS Notificateur (Firefox).

Je confirme le bug de Breizh (note que avec ma version de Firefox n’autorise pas à installer l’extention à partir de github, puisque elle est pas signée). Et je l’ai installée de addons.mozilla.com (hier)

Et je confirme aussi mais ce n’est pas le bug de Breizh, vu que ça ne marche plus du tout depuis un temps entre hier et 3 jours

J’essaye de voir ça dans le week end

<3

Il y a bien un problème en ce moment sur l’extension, j’essaye de voir ça en soirée, je viens de trouver d’où vient le problème (à priori maintenant je me retrouve avec des parseerror quand je get https://zestedesavoir.com/api/notifications/?page_size=30&ordering=-pubdate&Authorization=xxxx).

J’essaye de régler ça au plus vite, désolé du dérangement.

Je viens de pusher sur le fix sur le github (https://github.com/zestedesavoir/extensions-notificateurs/tree/master/Firefox).

La version devrait être acceptée assez vite depuis le marketplace (d’ici 2 jours maximum).

Des bisous.

Merci pour ta rapidité

Pas de soucis ;). Normalement si vous avez laissé la mise à jour auto, votre version devrait maintenant etre à jour sur firefox. Chez moi ça remarche

En effet, la mise à jour s’est faite. Et le bug que j’avais est résolu. Merci .

Super !

Le bouton ouvrir toutes les notifications est déjà prévu ouai.

Pour la séparation, je l’avais fait avant. Je l’ai enlevé parce que c’est chiant d’avoir des onglets (je devrais peut etre faire 2 zones). Du coup je ne sais pas trop; Mais déjà j’ai la version chrome à porter

Super, je viens de me rendre compte que c’était fixé chez moi aussi (en recevant une notif de cette discussion, sacrée mise en abîme).

Depuis la dernière mise à jour je ne reçois plus de notifications. En creusant un peu il semble que ça vient du fait que l’API renvoie du XML par défaut alors que tu voudrais du JSON.

En creusant un peu il semble que ça vient du fait que l’API renvoie du XML par défaut alors que tu voudrais du JSON.

Euh non, l’API renvoie bien du JSON par défaut (?).

En allant sur https://zestedesavoir.com/api/notifications/ j’ai du XML moi. J’ai regardé dans le fichier Firefox/notifier.js de la branche master (est-ce que c’est la bonne branche ?) et il y a l’air de manquer un petit ?format=json !

Après chez moi l’extension fonctionne bien donc je suppose que ça ne doit pas être la bonne branche du dépôt Git ?

En allant sur https://zestedesavoir.com/api/notifications/ j’ai du XML moi.

A priori, c’est ton navigateur qui ajoute par défaut l’header Accept, demandant du XML. Si tu supprimes cet header, par défaut, l’API va bien renvoyer du JSON.

Le format a changé, avant je récupérais bien du XML par défaut, mais depuis quelques temps l’API me donne du JSON, j’ai donc forcé au JSON pour que ça marche.

@Bat’, tu as pris la derniere version où ? Celle la fonctionne normalement => https://addons.mozilla.org/fr/firefox/addon/zds-notificateur/

Il me semble que je l’avait installé depuis le lien que tu as donné. En tout cas je l’ai réinstallé à partir de là. Je sais pas si ça marche par contre vu que j’ai pas reçu de notif entre temps.

o/

Ça fait longtemps que je n’avais pas remonté ce sujet. Je suis entrain de travailler sur la 2.3, avec en nouveauté, un bouton pour créer directement un nouvel MP comme le site, et un gros clean du code (merci @motet-a de m’avoir rappellé à quel point le code est sale).

2 petits screenshots :

Si vous voulez tester : https://github.com/AmarOk1412/extensions-notificateurs/tree/clean_firefox

\o

Au niveau de la compatibilité avec Firefox 57 ? C’est bon ?