cozy / coachCO2

CoachCO2 aims to raise user awareness about their carbon footprint, notably based on their transport data.
GNU Affero General Public License v3.0
7 stars 1 forks source link

Feat/ver 248 #418

Closed Merkur39 closed 6 months ago

Merkur39 commented 6 months ago

Cette PR corrige la disparition des sources dans le sélecteur sur la page des paramètres de l'application. En effet, dans le cas où un connecteur est déconnecté ou désinstallé, la source qui n'est autre que le compte(io.cozy.accounts) est supprimé.

En sachant cela, et comme nous souhaitons pouvoir continuer à sélectionner les noms des comptes avec leur trajets, nous devons plutôt nous servir de la propriété captureDevice présente sur tous les timeseries qui ont pour valeur le nom du compte d'où elles proviennent.

Ainsi, nous pouvons continuer à sélectionner le nom des comptes et afficher leurs trajets.

Le service ne réagissant qu'à la création d'un timeseries, nous avons besoin de créer un Job et de le déclencher une fois "manuellement" lorsque l'utilisateur arrive sur l'app. Ainsi il n'aura pas à attendre la création d'un nouveau trajet pour voir ses anciennes sources dans ses paramètres. (ce Job est conditionné à la non présence du nouveau paramètre accountsLists dans les settings de l'app créé par le service)

### 🐛 Bug Fixes

* Display all sources in AccountSelector
bundlemon[bot] commented 6 months ago

BundleMon

Files updated (1) Status | Path | Size | Limits :------------: | ------------ | :------------: | :------------: :white_check_mark: | app/coachco2.(hash).js
| 68.89KB (+127B +0.18%) | -
Unchanged files (3) Status | Path | Size | Limits :------------: | ------------ | :------------: | :------------: :white_check_mark: | vendors/coachco2.(hash).js
| 1.72MB | - :white_check_mark: | vendors-coachco2.(hash).(hash).min.css
| 38.65KB | - :white_check_mark: | app-coachco2.(hash).min.css
| 356B | -

Total files change +127B +0.01%

Final result: :white_check_mark:

View report in BundleMon website ➡️


Current branch size history | Target branch size history

Merkur39 commented 6 months ago

Cette PR ne doit plus se baser sur master mais sur la beta 0.18.0 => https://github.com/cozy/coachCO2/pull/419