CovidTrackerFr / vitemadose-front

Interface utilisateur de l'outil ViteMaDose
https://vitemadose.covidtracker.fr
Other
80 stars 51 forks source link

Utiliser la configuration firebase des applications mobile #199 #207

Closed Aymerik closed 3 years ago

Aymerik commented 3 years ago

Issue #199

C'est un premier jet. Le tout peut être encore amélioré.

Plusieurs points à noter :

  1. Le disclaimer n'est pas utilisé pour l'instant. Où doit-il être intégré ?
  2. Pour l'instant, j'ai utilisé une app perso de firebase car je ne connais pas les données de l'app VMD sur Firebase
  3. J'ai mis en brut les données d'"identification" de l'app dans Config.ts, pas sûr que ce soit une bonne idée. J'avais pensé à passer par un fichier .env mais je ne sais pas si c'est utile, dans tous les cas ces données seront visibles depuis le navigateur. Ça peut être intéressant pour bien séparer ces infos et pouvoir avoir une app firebase différente en local et en prod pour faire des tests éventuels. Bref j'ai pas su me décider, un retour sur le sujet serait intéressant

Et je suis preneur de toute remarque sur tout le reste bien sûr ! ;-)

Floby commented 3 years ago

Je serai pour utiliser moins de chose de la config firebase. Ce genre de fonctionnalité de remote config est utile sur mobile parce qu'il ne peuvent pas forcer de mise à jour des clients. sur le web on a pas ce problème. Complexifier la config, c'est bouger du code en dehors de la base de code vers un endroit ou ne peut pas tester.

Donc je serai pour se limiter aux trucx dont on connait l'utilité dès maintenant

fcamblor commented 3 years ago

@Aymerik je ferme ta PR au profit de #262 dans laquelle j'ai repris un certain nombre de tes idées (en les poussant un poil plus loin :-) )