feat: Gérer un état 'stopped' local en plus de l'état synchronisé #60
Conversation
| @@ -99,13 +100,11 @@ | |||
| }); | |||
|
|
|||
| document.getElementById("stop").onclick = async () => { | |||
| await browser.storage.sync.set({ stopped: true }); | |||
| displayStopStart(true); | |||
There was a problem hiding this comment.
La simplification minime dont je parlais: en effet, displayStopStart est appelé dans onChanged, donc je l'ai enlevé d'ici (et de dessous).
| @@ -111,7 +109,14 @@ | |||
| }); | |||
| } | |||
|
|
|||
| if (change.stopped) return await updateIconStatus(stopped); | |||
| if (change.stopped) { | |||
| await updateIconStatus(change.stopped.newValue); | |||
There was a problem hiding this comment.
la seconde partie du fix de l'icône
There was a problem hiding this comment.
Arf, j'ai déjà fixé tout ca dans #48
| @@ -10,7 +10,7 @@ | |||
| WORKING: "w", | |||
| }; | |||
|
|
|||
| async function updateIconStatus() { | |||
| async function updateIconStatus(stopped) { | |||
There was a problem hiding this comment.
la première partie du fix de l'icône
julienw
force-pushed
the
store-local-state-in-local
branch
from
8d1be38
to
c28b9c5
Compare
| stopped: false, | ||
| // Le booléan "stopped" est à la fois stocké localement et synchronisé. En | ||
| // effet, lorsqu'on arrive à booker un rdv dans un Firefox on veut arrêter les | ||
| // checks dans toutes les instances. Mais un simple clic sur le bouton ne doit |
There was a problem hiding this comment.
On est sûr de ça ? À mon avis il vaut mieux arrêter partout. Cas d'usage :
- je trouve un RDV sur Maia ou Keldoc
- je veux que ça s'arrête sur tous mes ordis
There was a problem hiding this comment.
Ah ben je pensais que tu étais d'accord d'après #56 (comment), c'est le but de ce PR à la base :-)
En fait je comprends pas bien l'intérêt de faire tourner l'extension sur plusieurs ordinateurs à la fois. Le problème est qu'elle est installée automatiquement lorsqu'on se connecte avec son FxAccounts sur une autre installation...
There was a problem hiding this comment.
Ah ben je pensais que tu étais d'accord d'après #56 (comment), c'est le but de ce PR à la base :-)
Je pensais que tu voulais tout mettre en local storage (ce qui me va toujours). Je n'aime pas trop le mélange des deux, ça rend ça moins clair. Si rien n'est synchro, ça me semble plus logique que tu doives l'arrêter partout.
There was a problem hiding this comment.
Mais du coup, ce cas d'usage:
- je trouve un RDV sur Maia ou Keldoc
- je veux que ça s'arrête sur tous mes ordis
ne sera plus supporté?
Et tu veux mettre les locations aussi en local, ou bien pour ça c'est OK que ce soit à la fois synchronisé en local ? Je vois que c'est déjà en partie en local pour pouvoir les afficher, mais c'est pas tout à fait clair pour moi vu que j'ai pas du tout suivi cette partie...
Désolé, j'essaie de bien comprendre ce qu'on veut faire :-)
(pas encore prêt pour merger, je veux faire plus de tests notamment pour la partie "synchro")
Fixes #56
J'ai finalement gardé un état synchronisé car je trouve utile d'arrêter la recherche sur toutes les instances de Firefox quand un rdv est booké.
En revanche il n'y a plus de bouton pour gérer cet état synchronisé, donc ça peut être pénible pour les personnes qui veulent trouver plusieurs rendez-vous (sauf à supprimer l'ensemble des données). Donc feedback bienvenu sur ce sujet.
J'en ai profité pour corriger l'icône qui ne changeait pas de couleurs en fonction de l'état ;-) et un tout petit peu de simplifications par ci par là sur les changements d'UI lorsqu'un état change.