zestedesavoir / extensions-notificateurs Goto Github PK

Notificateur pour Zeste de Savoir (Chrome, Firefox, Mac)

License: MIT License

JavaScript 67.21% CSS 14.54% HTML 18.25%

extensions-notificateurs's Introduction

Qu'est-ce que Zeste de Savoir ?

Zeste de Savoir est un site internet communautaire dédié au partage de la connaissance pour tous. Il est propulsé par le framework Django et Python 3.

Zeste de Savoir était à l'origine un fork de Progdupeupl (voir le dépôt Git).

Notre projet technique

Notre projet technique est constitué de plusieurs éléments :

zds-site, le cœur du projet technique ;
zmarkdown, notre moteur Markdown amélioré pour les besoins du site ;
ansible-zestedesavoir, notre outil pour déployer Zeste de Savoir sur un serveur de production ;
latex-template, notre gabarit LaTeX pour générer des fichiers PDF ;
extensions-notificateurs, nos extensions de navigateur pour recevoir des notifications.

Contribuer à Zeste de Savoir

Notre documentation technique devrait vous être utile pour bien appréhender notre projet.

Merci de prendre connaissance du Code de Conduite de Contributeurs et de le respecter pour garder ce projet ouvert et accueillant !

Nous contacter

N'hésitez pas à discuter avec nous sur le forum Dev Zone de Zeste de Savoir ou sur le canal #dev-de-zds de notre Discord !

Installation

Cette procédure détaillée devrait vous permettre d'installer le projet en autonomie. Si vous rencontrez des difficultés, n'hésitez pas à nous contacter !

Conseils pour débuter

extensions-notificateurs's People

Contributors

Stargazers

Watchers

Forkers

axylium s-afchar situphen eskimon codewire amarok1412 drulac asakumaku arobase-che stallaf tuxmania

extensions-notificateurs's Issues

Chrome Tailor ?

Je viens de tomber sur cet article de Korben et je me suis dit que ça pourrait être une façon simple de maintenir l'extension Firefox avec une seule base de code, non ?

Après j'ai plus les API en tête, mais de mémoire on en utilise pas énormément, si ?

Poke @AmarOk1412 @Eskimon

Utiliser des set/get

Ça permettrait d'éviter certaines méthodes intermédiaires et en même temps d'automatiser certaines choses.

Par exemples, un set notifications(list) permettrait en même temps de les enregistrer mais aussi de mettre à jour le badge de l'extension.

Et ça rendrait le code plus propre, tant qu'à faire…

Porter la version Webextension sur Chrome

Le code de la version Firefox est compatible avec Chrome, par contre il reste pas mals de trucs à modifier.

La version Firefox ne récupère pas toutes les notifications

Les plus vieux sujets ne sont pas récupérés, il faut agrandir la taille de la page récupérée via l'API + les notifs staff et MPs

Accès à toutes ses notifications & à ses paramètres

Quand j'essaye d'accéder à toutes mes notifications via l'extension, je suis redirigé vers une erreur 404 ! En fait, ton URL n'est pas bonne, il manque un « S » à forum. ;-)

C'est : « http://zestedesavoir.com/forums/notifications/ » et non « http://zestedesavoir.com/forum/notifications/ »

De même pour les paramètres du profil. Tu rajoutes le pseudo à la fin de URL, or, il n'y est pas normalement. ;-)

(Sinon, j'adore !)

La notification reste si ouverte dans un nouvel onglet

Lorsque j'ouvre une notification dans un nouvel onglet (clique milieu ou clique droit -> nouvel onglet), elle reste présente et le compteur ne diminue pas.
Un clique gauche ouvre aussi dans un nouvel onglet, mais la notification disparaît immédiatement.

Réponse topic, pas de notif bureau

Lorsque quelqu'un répond à un topic suivi, je n'ai pas de notification bureau.
En revanche le chiffre augmente sur l'icône et la notif est bien visible dans la bulle qui apparaît lors d'un clique sur l'icône "Zds Notificateur".

Je précise que tout marche pour les MP

Future break-change de l'api

Une version future (indéterminée) de l'api va entraîner une break change

Une break change est prévu pour api/mps/, participants: (2) [1, 2] ne sera plus un array d'ids mais d'objects on aura participants: (2) [{...}, {...}] dans l'idée de la clé sender de api/notification/.

La PR liée à ce correctif est : zestedesavoir/zds-site#5787
Conversation liée : https://discordapp.com/channels/547038592892010516/547038766506967041/708257530542817340

Double barre oblique dans les liens

Je viens de recevoir une notification avec deux barres obliques dans le lien :

https://zestedesavoir.com//billets/2173/le-premier-jour-du-reste-de-ma-vie/?page=1#p167009

Le lien fonctionne toujours (au moins sur Firefox desktop mais je suppose que c'est pareil sur la majorité des navigateurs) donc ce n'est pas bloquant mais c'est à corriger :)

Notif. nouveaux articles & tutos

Ce serait cool d'être notifié des nouveaux articles et nouveaux tutos publiés.

En stockant l'info du dernier item c'est facile de savoir ce qui est nouveau et la home reprend l'ensemble des infos nécessaires : notifications, MP, articles, tutoriels…

Notifications Staff

Une fonctionnalité intéressante serait d'afficher aussi les notifications propres au Staff.

Ça risque d'être lourd, mais ça pourrait accélérer (et simplifier) le boulot des modérateurs par exemple…

On vire tout et on r'commence!

@Eskimon Tu vas pas aimer, mais je pense que ça serait pas mal d'un peu tout revoir... ✌️

En gros, au programme:

Mise en place d'un workflow un peu plus moderne, avec:
- Un task-runner (Gulp, grunt ?)
- JSHint
- Des tests unitaires
- Un préprocesseur CSS
Utilisation d'une framework qui permet du data-binding two-ways
- Laquelle ? J'ai une préférence pour Angular \o/
- Pourquoi ? Parce que on devrait pourvoir correctement mettre à jour la popup alors qu'elle est ouverte
Stabiliser un peu tout le bordel, parce que bon... C'est un peu bancal pour le moment ^_^

Alors bon, quand je dis "on vire tout", c'est pas exactement ca... On va pas se retaper un rewrite complet du background, avec le parsing et tout... 😸 C'est surtout pour la popup et les pages d'options et autre...

Voila voila, moi je risque de revenir d'ici quelques jours, donc @Eskimon, tu as le temps de te préparer psychologiquement :D (btw, si tu veux qu'on en parle, je suis dispo ce soir o/ )

Publier sur le Chrome Web Store ?

fd93e61 n'est pas disponible sur la version Chrome installé en passant par le Chrome Web Store. :|

Popup vide à la première ouverture

Bug vu et revu : la popup indique qu'il n'y a pas de notifications à la première ouverture mais se remplit bien à la 2ème (ou 3ème).

Impossible d'accéder à la page des options

j'ai essayé d'aller sur la page des options mais le lien est mort. Sera-ce résolu dans une prochaine version ?

Badge pas à jour

Quand une notif est supprimée, le badge n'est pas mis à jour et reste donc avec un nombre faux.

Crash du système

Salut,

Je viens rapporter un bug ennuyant puisque l'extension fait littérelament crasher mon système et malheureusement, je ne vais pas pouvoir donner beaucoup d'information.

Procédure :

Recevoir une notification.
Cliquer dessus.
Se rendre compte que le pannel n'est jamais ouvert et que le système est crashé.

Attention, le bug ne survient pas systématiquement. En fait, il est plutôt rare mais il arrive assez régulièrement pour que je vienne le rapporter directement sur le dépôt technique.

Environnement :

Système : OS X 10.9.5 (Mavericks).
Navigateur : Google Chrome Version 51.0.2704.106 (64-bit)

Refresh manuel

Amélioration possible : pouvoir rafraîchir manuellement, par exemple en ajoutant une option dans le menu de l'extension :

Au clic droit sur l'icône — à voir si possible, je trouve pas de doc sur le sujet
Dans la popup, via un bouton dédié (oblige à ouvrir la popup, pas forcément un comportement souhaité, surtout si l'icône est inactive)

Meilleur rafraichissement

Je propose un commit pour régler 1 bug connu :

bug visuel de l'affichage de ses propres notifs du à un rafraichissement trop rapide

En plus il permet de faire les refresh toutes les X secondes et non au clic

Les liens de vos profils mènent vers des 404

Les liens de vos profils à toi et @sandhose, mènent vers des erreurs 404. Il manque le « voir » dans l'URL. Car c'est : « http://zestedesavoir.com/membres/voir/Eskimon/ » et non « http://zestedesavoir.com/membres/Eskimon/ ». ;-)

Double notif sonore MP

Quand on reçoit un nouveau MP, il y a un son.

Mais quand j'ouvre la liste pour voir ce qu'il y a, j'entends à nouveau le son. Du coup il y a peut-être un filtre à mettre en place quelque part.

P.S. : promis, dès que j'ai un peu de temps je me mets aux extensions Chrome pour pouvoir filer un vrai coup de main ;)

Rendre possible l'exploitation sur des forks

Ne serait-il pas heureux de pouvoir personnaliser l'URI du site, afin de pouvoir l'utiliser sur des éventuels forks du projet, sans devoir pour autant distribuer une nouvelle extension ?

Toujours une notification

Même lorsque j'ai lus tous les sujets ayant reçu de nouvelles réponses, le notificateur continue de m'afficher un "1".

Cependant, si je clique sur le notificateur, le compteur disparaît et la fenêtre ne m'affiche aucune notification.

Firefox 13 : l'extension n'est plus acceptée par Firefox

Hello,

Je ne sais pas si les extensions sont toujours maintenues, mais aujourd'hui j'ai eu droit à une petite surprise de la part de Firefox :

Apparemment, c'est lié au fait que l'extension n'est pas signée. Firefox propose bien un moyen de la réactiver, mais il faut pour ça désactiver totalement la protection contre les extensions non signées dans about:config, ce qui n'est pas forcément idéal…

Icon Firefox pas assez visible

On ne voit pas qu'on une notification, il faudrait du rouge comme sur le site !

Remarques orthographiques ;)

qui vous averti -> qui vous avertit
à interval régulier -> à intervalle régulier
vous les affichent -> vous les affiche
la barre d'addresse -> la barre d'adresse
on prenderas en compte -> on prendra en compte
tout Pull request -> toute Pull request (féminin, il me semble)
horifié par le code -> horrifié par le code

My 2 cents.

Ré-écriture du README.md

Bonjour à tous,

Suite à plusieurs nouveaux projets (ClemNotificateur, et l'extension pour Firefox) pour les notificateurs de Zeste De Savoir, le README.md n'a toujours pas été mis à jour.

Ce qu'on pourrait faire c'est de le mettre à jour juste pour notre notificateur. Et ensuite le mettre en ligne.

Qu'en pensez vous ?

Toutes les notification renvoi sur la mauvaise url

Le bouton "toutes les notifications renvois vers l'url https://zestedesavoir.com/forums/notifications/
alors qu'il devrai renvoyer vers https://zestedesavoir.com/notifications/
une url est mal écrite dans le code
Je suis avec le navigateur vivaldi et opera, je n'ai pas testé avec firefox

Il faudrait voir ce que Piwit peut faire, sinon je m'y collerai un jour.

Icône inactive

Depuis ce midi l'icône est devenue inactive : transparente et aucune réaction au clic. Par contre le compteur augmente bien comme il faut.

Du coup je suis bien averti quand il y a du nouveau, mais je dois encore aller à la main sur ZdS…

Un petit aperçu…

Fix graphiques

Changer la police pour celle de Zds
Agrandir un peu la pop-up
overflow:hidden pour le pseudo si trop grand