search

leximpact / leximpact-client

Simuler l'impact des réformes socio-fiscales en moins d'une minute
https://leximpact.an.fr
GNU Affero General Public License v3.0
3 stars 0 forks source link

Intégrer la DF #87

Closed LoicPoullain closed 4 years ago

LoicPoullain commented 4 years ago

https://trello.com/c/UPw8nU5Z/423-ajouter-df-components

Cette PR affiche la DF avec ?df.

Comme le serveur ne renvoie pas encore de données, les données utilisées sont celle de la DSR.

sandcha commented 4 years ago

@DorineLam Ce sera cette PR bientôt.

LoicPoullain commented 4 years ago

Note à moi-même : avant de fusionner, mettre les résultats DF à 0 (et non égaux à la DSR) pour ne pas avoir des "tendances" incorrectes (DSR compté en double)

LoicPoullain commented 4 years ago

@DorineLam : demande de revue.

DorineLam commented 4 years ago

@LoicPoullain proposition de modifs du texte de loi pour rendre plus clair qu'on parle uniquement l'écrêtement et pour rendre plus accessible le texte :

Capture d’écran 2020-09-24 à 14 25 39 4
sandcha commented 4 years ago

@LoicPoullain, Au cas où, au lancement du client, il y a déjà une valeur d'amendement dans la zone impact. Je vois ceci en première ligne du tableau des strates sans avoir défini d'amendement :

Screen Shot 2020-09-30 at 14 37 36

Testé en local avec cette PR client et la PR leximpact/leximpact-server#37.

Par ailleurs, tu indiques que les valeurs en dur sont celles de la DSR mais cela semble différent.

LoicPoullain commented 4 years ago

@sandcha ça devrait pas arriver. Je regarde ça.

LoicPoullain commented 4 years ago

@sandcha c'est corrigé. Les valeurs utilisées par défaut étaient celles de la DSR.