Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Ajoute ou renomme les libellés des paramètres #1669

Merged
merged 62 commits into from
Oct 7, 2021
Merged

Ajoute ou renomme les libellés des paramètres #1669

merged 62 commits into from
Oct 7, 2021

Conversation

DorineLam
Copy link
Contributor

@DorineLam DorineLam commented Sep 7, 2021 •
edited by sandcha
Loading

  • Changement mineur.
  • Périodes concernées : toutes.
  • Zones impactées :
    • parameters/demographie
    • parameters/epargne
    • parameters/fonc/indem_resid et parameters/fonc/supplement_familial
    • parameters/impot_revenu
    • parameters/marche_travail/salaire_minimum
    • parameters/prelevements_sociaux
    • parameters/prestations/prestations_familiales/af
  • Détails :
    • Améliore les libellés des paramètres (accents, formulations ...)
    • Modifie les formulations des libellés des paramètres de plafond du QF de l'impôt sur le revenu (en utilisant les formulations du site economie.gouv / et celles de l'IPP)

@DorineLam
Copy link
Contributor Author

@Sasha-Laniece dans le commit précédent, je propose une modification de l'UX-name "Plafond SS" vers "Plafond de la sécurité sociale", il me semble que ce n'est pas suffisamment compréhensible à partir du dossier parent "Cotisations de Sécurité sociale du régime général". Qu'en penses-tu ?

@Sasha-Laniece
Copy link
Contributor

@Sasha-Laniece dans le commit précédent, je propose une modification de l'UX-name "Plafond SS" vers "Plafond de la sécurité sociale", il me semble que ce n'est pas suffisamment compréhensible à partir du dossier parent "Cotisations de Sécurité sociale du régime général". Qu'en penses-tu ?

Ça me semble pertinent, car les abréviations sont peu claires et le but du ux_name est la clarté

@DorineLam DorineLam marked this pull request as ready for review September 29, 2021 06:26
@DorineLam
Copy link
Contributor Author

@HAEKADI est-ce que je peux te demander de l'aide pour résoudre les tests de cette PR ? et pour la review la PR ?

@HAEKADI
Copy link
Contributor

HAEKADI commented Sep 29, 2021

@DorineLam C'est la petite coquille ci-dessus qui pose problème ici :)

@DorineLam
Copy link
Contributor Author

@DorineLam C'est la petite coquille ci-dessus qui pose problème ici :)

Merci !

@HAEKADI HAEKADI mentioned this pull request Sep 30, 2021
Merged
@HAEKADI
Copy link
Contributor

HAEKADI commented Sep 30, 2021

@DorineLam Même avec la corrections des typos, le problème persiste parce que le build n'est pas mis à jour.
J'ai ouvert une PR #1685 qui va résoudre ce problème une fois mergée :)

@DorineLam
Copy link
Contributor Author

Merci beaucoup. @HAEKADI

@DorineLam @Sasha-Laniece
Ajoute les accents
df6243c
@DorineLam @Sasha-Laniece
Suppression de l'unité dans la description
96c37f3 
Ce commit supprime l'unité de la description, l'unité devant figurer de préférence dans le champ "unit"
@DorineLam @Sasha-Laniece
Enlève les majuscules à SMIC
1aecb52 
SMIC est écrit Smic dans service-public.fr : https://www.service-public.fr/particuliers/vosdroits/F2300
@DorineLam @Sasha-Laniece
Ajout d'une description
cb8124d
@DorineLam @Sasha-Laniece
Ajout d'une description
e83fee5
@DorineLam @Sasha-Laniece
Propose modification UX_name
a64a43a
@DorineLam @Sasha-Laniece
Modifie l'ux-name
9297f2d 
Ajoute plafond pour plus de clarté
@DorineLam @Sasha-Laniece
Modifie Ux-name
50ca9ae 
Ajoute plafond pour plus de clarté
@DorineLam @Sasha-Laniece
Ajoute majuscule à la description
817524d 
Ajoute majuscule à Smic => typographié ainsi dans service-public.fr
@DorineLam @Sasha-Laniece
Ajoute Ux-name
d0aebc6
@DorineLam @Sasha-Laniece
Ajoute ux_name
49ecdeb
@DorineLam @Sasha-Laniece
Ajoute ux_name
17cc3a7
@DorineLam @Sasha-Laniece
Ajoute ux_name
58255b9
@DorineLam @Sasha-Laniece
Ajoute ux_name
a3a8fa2
DorineLam and others added 26 commits October 7, 2021 19:38
@DorineLam @Sasha-Laniece
Corrige failed test_python, retire les guillemets
e860ef0
@DorineLam @Sasha-Laniece
Propose "Plafond" plutot que "Maximum"
58f6085 
Pour harmoniser avec les autres paramètres
@DorineLam @Sasha-Laniece
Propose "plafond" plutôt que "limite"
e6f2d6f 
Pour harmoniser avec les autres paramètres
@DorineLam @Sasha-Laniece
Ajoute majuscule au début
d8224c7
@DorineLam @Sasha-Laniece
Corrige orthographe
4360218
@DorineLam @Sasha-Laniece
Ajout description
d502b4e
@DorineLam @Sasha-Laniece
Propose de préciser le titre
29ca901 
Reprend le même titre que celui utilisé par economie.gouv.fr : https://www.economie.gouv.fr/particuliers/quotient-familial
@DorineLam @Sasha-Laniece
Change le titre du paramètre
d5c676d 
Il me semble que titre "Mariés pacsé" ne correspond pas du tout au contenu du paramètre. Je fais peut-être une erreur, mais ce plafond me semble être celui qui est attribué dans le cas d'une demi-part supplémentaire donnée en raison d'une personne à charge, indifféremment du fait que le cas type soit un couple ou un célibataire. Je m'appuie pour cela sur cet article (que j'ai ajouté dans les références => https://www.economie.gouv.fr/particuliers/quotient-familial) @sandcha peux-tu vérifier ma proposition ? (Les prochains commits vont aller dans le même sens que celui-ci.
@DorineLam @Sasha-Laniece
Change le titre du paramètre
fc19cdd 
Ici aussi comme dans le commit suivant : 94881bd je propose de changer complètement le libellé.

Le plafond de 938 euros semble plus lié au personne ayant élevé seules un enfant pendant plus de 5 ans qu'en raison d'un statut marital => https://www.economie.gouv.fr/particuliers/quotient-familial) @sandcha peux-tu vérifier ma proposition ?
@DorineLam @Sasha-Laniece
Change le titre
6b37c4e 
Idem que pour les deux commits précédents. @sandcha peux-tu jeter un coup d'oeil ?
@DorineLam @Sasha-Laniece
Change le titre de la description
03f1a41 
Idem que pour les 3 commits précédents.
@DorineLam @Sasha-Laniece
Changement titre du paramètre
f54a2d5 
Idem que pour les 4 commits précédents
@DorineLam @Sasha-Laniece
Ajoute une descritpion au dossier
92aad74
@DorineLam @Sasha-Laniece
Supprime la précision qui est désormais dans le titre (dossier supéri…
86abe2e 
…eur)
@DorineLam @Sasha-Laniece
Corrige faute de frappe
2835797
@DorineLam @Sasha-Laniece
Précise le périmètre du plafonnement dépasse le cadre "mariés ou pacsés"
8c2303c 
Ce commit précise le commit 94881bd qui proposait un nouveau libellé pour le paramètre maries_ou_pacses
@DorineLam @Sasha-Laniece
Ajoute l'élément sur lequel le plafond est appliqué
e0d94fc
@DorineLam @Sasha-Laniece
Ajoute l'élément sur lequel le plafond est appliqué
b4ce24c
@DorineLam @Sasha-Laniece
Simplifie le libellé et ajoute l'élément sur lequel est appliqué le p…
7bad068 
…lafond
@DorineLam @Sasha-Laniece
Corrige la formulation proposée (augmentation du plafond => réduction…
a47e2e2 
… complémentaire)

Ce commit propose de reprendre la formulation utilisée par impôt.gouv https://www.impots.gouv.fr/portail/particulier/questions/je-suis-ancien-combattant-ou-veufveuve-dancien-combattant-cela-modifie-t-il et par l'IPP https://www.ipp.eu/baremes-ipp/impot-sur-le-revenu/calcul_impot_revenu/plaf_qf/
@DorineLam @Sasha-Laniece
Retire le point final
dc606a8
@DorineLam @Sasha-Laniece
Corrige la formulation proposée (augmentation du plafond => réduction…
12cb7e8 
… complémentaire)
@DorineLam @HAEKADI @Sasha-Laniece
Update openfisca_france/parameters/impot_revenu/prelevement_forfaitai…
d3c1c9d 
…re_unique_ir/index.yaml

Co-authored-by: Hajar AIT EL KADI <48837850+HAEKADI@users.noreply.github.com>
@HAEKADI @Sasha-Laniece
Corrige typo
c188cd5
@Sasha-Laniece
Ajoute label pour le noeud 'cotsoc'
2a2ecfb
@Sasha-Laniece
Ajoute l'unit (PSS)
f0c0841
@Sasha-Laniece Sasha-Laniece merged commit 57ed497 into master Oct 7, 2021
@Sasha-Laniece Sasha-Laniece deleted the libelles-parametres branch October 7, 2021 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@HAEKADI HAEKADI HAEKADI left review comments

@Sasha-Laniece Sasha-Laniece Sasha-Laniece approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@DorineLam @Sasha-Laniece @HAEKADI