Skip to content
Snippets Groups Projects
Commit 7ffff15d authored by sandcha's avatar sandcha
Browse files

Remove temporary notes from README_PLF

parent 45d8aa9b
No related branches found
No related tags found
1 merge request!156Désactive le PLF de l'application IR
Pipeline #3413 passed
`leximpact-client` échange par API Web avec `leximpact-server`.
Les deux dépôts disposent d'éléments de configuration qui déterminent l'état initial de ces applications.
Néanmoins, pour le couple (`leximpact-client`, `leximpact-server`) la logique générale veut que ce soit le client qui sache si le PLF est activé.
> Aux débuts de ces applications (pré-PLF 2021), l'activation du PLF était décidée par `leximpact-server`.
## Sans PLF
Etat :
base
amendement
Réponse /calculate/compare :
avant
apres
/ir ou /dotations
### Bug des tirets
Intervient dans 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 (à confirmer).
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).
## Avec PLF
Etat :
base
amendement
plf
Réponse /calculate/compare :
avant
apres
plf
> redux/reducers/results/total-pop.ts - foyersFiscauxTouches
avant_to_plf
avant_to_plf
plf_to_apres
`leximpact-client` connaît potentiellement 3 lois :
* la loi en vigueur, dite `base` dans l'état de l'application, `avant` dans les requêtes à l'API Web,
* la loi réformée par un usager de l'application, dite `amendement` dans l'état de l'application, `apres` dans les requêtes à l'API Web,
* et, lorque le PLF est activé, la loi réformée par un PLF, dite `plf` dans l'état de l'application et les requêtes à l'API Web.
### Calcul sur cas types
......@@ -69,28 +39,29 @@ Et, du state aux properties d'une SimpleCard : components/ir/cartes-impact/simpl
La zone impact d'une carte de cas type (bas de la carte) : components/ir/cartes-impact/simple-card/impact-impots.tsx - SimpleCardImpactImpots
results.base.ir.state
results.amendement.ir.state
results.plf.ir.state
Reducers dédiés : redux/reducers/results
Calcul d'une requête : redux/actions/simulations/simulate-cas-types.ts
### Calcul sur population
Les valeurs au chargement de l'application sont configurées dans le fichier `redux/reducers/results/total-pop.ts`
> PLF :
> IR components/ir/cartes-impact/carte-etat/CarteEtat.tsx (e40d153c80fdd7c158b0e0b24bcfc945b41b08b9) ?
> dotations components/dotations/articles/montant-dsr-dsu/majoration-minoration-section/majoration-minoration-panel/MajorationMinorationPanel.tsx (+ 086e8404e8ea0a5749e43f2ef33b23ecc54c29ce) ?
### 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é.
## 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).
Le RootState est l'interface de l'état global redux. Elle est déduite des réduceurs. De cette manière, on peut déduire des bugs en amont en l'utilisant à chaque fois dans les mapStateToProps. ([src](https://github.com/leximpact/leximpact-client/pull/46))
Le root state est défini ici : redux/reducers/index.ts
L'état est construit ici : redux/make-application-state.ts
Les interfaces des états redux se situent maintenant dans le dossier redux/reducers avec leur réduceur associé ([src](https://github.com/leximpact/leximpact-client/pull/46)). = Les interfaces d'action se situent maintenant conjointement avec leur fonction créatrice.
Le format de ce state est défini ici (selon la partie gauche, la réforme paramétrique) : redux/reducers/parameters/interfaces/ir-state.ts
Interface total-pop : redux/reducers/results/total-pop.ts
L'arborescence des états de l'application est contenue dans un store ([en savoir plus sur un Redux store](https://redux.js.org/api/store))
* React via React Developer Tools [plugin chrome](https://chrome.google.com/webstore/detail/react-developer-tools/fmkadmapgofadopljbjfkapdkoienihi)
......@@ -101,10 +72,15 @@ L'arborescence des états de l'application est contenue dans un store ([en savoi
/public : les éléments statiques de l'application
/redux : la gestion de l'état de l'application ; pour en savoir plus [Redux Fundamentals...: State, Actions, and Reducers](https://redux.js.org/tutorials/fundamentals/part-3-state-actions-reducers)
## TODO
Vérifier log : Can't write cookie on domain *.leximpact.an.fr
Réactiver le linting en CI (...8b01700258b106ab01d04930f9e37db94b7902cc) ?
Dispatcher le contenu du document 2020 : https://cloud.leximpact.dev/index.php/apps/onlyoffice/4060?filePath=%2FLexImpact_2020%2FProduit%20dotations%2FElements%20graphiques%20cartographie%2FAm%C3%A9liorations%20front%20_%20cartographie.docx ?
> Population, en local, prendre l'URL /connection/token et l'ajouter à [localhost](http://localhost:9001/ir) sans ir/
### Bug des tirets
Intervient dans 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).
plf à ajouter aux déciles dans redux/reducers/results/total-pop.ts comme ici https://github.com/leximpact/leximpact-client/pull/104/files ?
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment