From 04f6bae74d5edd74a4f7d0c42a93cc854d3d9636 Mon Sep 17 00:00:00 2001
From: sandcha <sandcha@users.noreply.github.com>
Date: Fri, 2 Sep 2022 17:37:34 +0200
Subject: [PATCH] Explicit PLF (de)activation steps as listed for current
MRgits
---
README_PLF.md | 31 +++++++++++++++++++++++++++----
1 file changed, 27 insertions(+), 4 deletions(-)
diff --git a/README_PLF.md b/README_PLF.md
index 037ab066..f03c8c48 100644
--- a/README_PLF.md
+++ b/README_PLF.md
@@ -4,6 +4,8 @@ Pour l'[application IR](http://leximpact.an.fr/ir) et l'[application dotations](
Cette documentation présente les éléments à mettre à jour pour activer et désactiver le mode `PLF`.
+> ⚠️ Cette documentation n'est pas exhaustive et s'apparente à ce stade à une prise de notes.
+
## Logique d'activation & désactivation du PLF
`leximpact-client` échange par API Web avec `leximpact-server`.
@@ -57,12 +59,32 @@ La zone impact d'une carte de cas type (bas de la carte) : components/ir/cartes-
Reducers dédiés : redux/reducers/results
Calcul d'une requête : redux/actions/simulations/simulate-cas-types.ts
-
### Calcul sur population
Avant, cela était défini dans redux/reducers/results/total-pop.ts
Maintenant, c'est récupéré de la requête au serveur lorsque celui-ci a le PLF activé.
+
+### Désactivation du PLF
+
+Pour désactiver le PLF, contrairement à `leximpact-server`, pour `lex-impact-client` le `.env` de l'application reste inchangé.
+
+Mais pour l'IR, il s'agit de :
+* Mettre à jour l'état par défaut `BASE_IR_DEFAULT_STATE` défini dans `./redux/reducers/parameters/base/ir.ts`
+* Vérifier la cohérence des montants sur les déciles de population définis dans `redux/reducers/results/total-pop.ts`
+* S'assurer de la cohérence du message d'information sur la carte budget `components/ir/cartes-impact/carte-etat/CarteEtat.tsx`
+* Mettre à jour le reducer en cas de `SIMULATE_CAS_TYPES_SUCCESS` défini dans `redux/reducers/results/plf/ir.ts`
+
+### Activation du PLF
+
+Pour activer le PLF, contrairement à `leximpact-server`, pour `lex-impact-client` le `.env` de l'application reste inchangé.
+
+Mais pour l'IR, il s'agit de :
+* Mettre à jour l'état présentant le PLF `PLF_IR_DEFAULT_STATE` défini dans `redux/reducers/parameters/plf/ir.ts` et qui diffère désormais de `BASE_IR_DEFAULT_STATE`
+* Vérifier la cohérence des montants sur les déciles de population définis dans `redux/reducers/results/total-pop.ts`
+* S'assurer de la cohérence du message d'information sur la carte budget `components/ir/cartes-impact/carte-etat/CarteEtat.tsx`
+* Mettre à jour le reducer en cas de `SIMULATE_CAS_TYPES_SUCCESS` défini dans `redux/reducers/results/plf/ir.ts`
+
## Debug
Un principe directeur de l'application est que l'interface graphique affichée dépend d'un état de l'application. Cet état est décrit dans [cette section du README](https://git.leximpact.dev/leximpact/leximpact-client/-/blob/master/README.md#organisation-de-létat-de-lapplication).
@@ -90,13 +112,14 @@ Des outils de développement facilitent grandement le debug de l'application com
#### Bug des tirets - Sur les cas types, pourquoi les valeurs sont-elles remplacées par des tirets ?
-Intervient dans components/common/articles-inputs/values/Values.tsx
+Au chargement de l'interface IR, l'impôt et le nombre de parts des cas types n'est pas représenté par un nombre mais par un `-`.
+Ceci intervient dans le composant components/common/articles-inputs/values/Values.tsx
+
Lorsque le PLF est désactivé du côté de leximpact-server seulement alors, on passe de :
`SimpleCardImpactImpots props {index: 5, amendement: 690, base: 756, isFetching: false, plf: 690, …}`
à `SimpleCardImpactImpots props {index: 5, amendement: undefined, base: undefined, isFetching: true, plf: undefined, …}`.
-Lié à la présence du `PLF_PATH` seulement.
-Le client nécessiterait toujours la présence du PLF pour fonctionner (ou une modification de son code). Ceci est en partie apparu avec [cette PR](https://github.com/leximpact/leximpact-client/pull/104).
+Ceci serait lié à la présence du `PLF_PATH` seulement. Le client nécessiterait toujours la présence du PLF pour fonctionner (ou une modification de son code). Ceci est en partie apparu avec [cette PR](https://github.com/leximpact/leximpact-client/pull/104).
#### Bug npm run dev - En mode dev ma modification n'est pas visible, que faire ?
--
GitLab