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 les aides de la ville de Antony #120

Merged
merged 14 commits into from
Nov 6, 2024
Merged

Ajoute les aides de la ville de Antony #120

merged 14 commits into from
Nov 6, 2024

Conversation

frtomas
Copy link
Contributor

@frtomas frtomas commented Nov 23, 2021

Ajoute 5 nouvelles aides pour la ville d'Antony

  • antony_aide_depart_sejour_adapte
  • antony_bourse_communale
  • antony_bourse_conservatoire
  • antony_bourse_famille_nombreuse
  • antony_noel_pour_tous

@frtomas frtomas changed the title Msa ajout aides antony Ajoute les aides de la ville de Antony Nov 23, 2021
@frtomas frtomas requested review from mtifarine and sandcha November 23, 2021 09:25
@frtomas frtomas force-pushed the msa_ajout_aides_antony branch from aed3d31 to cb3040a Compare November 24, 2021 09:55
@mtifarine mtifarine force-pushed the msa_ajout_aides_antony branch from f7829e0 to 04d3138 Compare November 30, 2021 15:09
Kout95
Kout95 suggested changes Jan 11, 2022
View reviewed changes
Copy link
Contributor

@Kout95 Kout95 left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Est-il possible d'avoir des références pour chaque aide afin de vérifier les calculs svp ? :)

guillett
guillett approved these changes Oct 12, 2022
View reviewed changes
Copy link
Member

@guillett guillett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je ne veux pas bloquer le merge de cette PR mais il me semble qu'il n'y a pas eu de relecture en interne à l'équipe du PNDS.

Il n'y aucune référence législative et le texte de la PR ne donne aucun élément de contexte permettant de vérifier/valider les implémentations.

Les textes en français, champs label, n'ont pas été relus, il manque des mots ou des accents.

Plus à la marge, certaines valeurs en dur pourraient être mises dans des paramètres.

openfisca_france_local/communes/antony/antony_noel_pour_tous.py Outdated Show resolved Hide resolved
openfisca_france_local/communes/antony/antony_bourse_famille_nombreuse.py Outdated Show resolved Hide resolved
openfisca_france_local/communes/antony/antony_bourse_famille_nombreuse.py Outdated Show resolved Hide resolved
openfisca_france_local/communes/antony/antony_bourse_conservatoire.py Outdated Show resolved Hide resolved
openfisca_france_local/communes/antony/antony_aide_depart_sejour_adapte.py Outdated Show resolved Hide resolved
tests/communes/antony/aide_bourse_conservatoire.yml Outdated Show resolved Hide resolved
tests/communes/antony/aide_bourse_conservatoire.yml Outdated Show resolved Hide resolved
tests/communes/antony/aide_bourse_famille_nombreuse.yml Outdated Show resolved Hide resolved
tests/communes/antony/aide_bourse_famille_nombreuse.yml Outdated Show resolved Hide resolved
tests/communes/antony/aide_depart_sejour_adapte.yml Outdated Show resolved Hide resolved
@Allan-CodeWorks
Copy link
Contributor

Bonjour,

Cette PR est ouverte et inactive depuis longtemps, est-ce qu’elle est encore pertinente?
Merci pour votre retour.

@frtomas
Copy link
Contributor Author

frtomas commented May 9, 2023

Cette PR est à priori bien toujours d'actualité, nous sommes en train de dépiler progressivement les vieilles PR qui traînent depuis un certain temps

@frtomas frtomas force-pushed the msa_ajout_aides_antony branch from 18136c3 to 23ecdb1 Compare July 11, 2023 14:36
@jenovateurs jenovateurs mentioned this pull request Oct 28, 2024
Closed
@jenovateurs jenovateurs requested a review from Shamzic October 30, 2024 17:22
frtomas and others added 12 commits November 6, 2024 13:56
@frtomas @Shamzic
Ajoute les parametres et l'eligibilite de residence des aides Antony
05ad6ce
@frtomas @Shamzic
Ajoute les aides de la ville de Antony
9e13c13
@mtifarine @Shamzic
ajout des cas de tests pour les aides de Antony
a2b4ab2
@frtomas @Shamzic
Ajoute une condition a la comparaison des ressources M-1 à m-3 et N-1
9fa65f1
@frtomas @Shamzic
Modifie les ressources prises en compte pour comparaison avec les 3 d…
f020961 
…erniers mois
@mtifarine @Shamzic
corrige les tests de la bases ressources des aides Antony
1dda563
@frtomas @Shamzic
Corrige les libéllés des tests
fc2ec42
@frtomas @Shamzic
Ajoute de nouveaux paramètres pour la ville de Antony
0586fc6
@frtomas @Shamzic
Utilise les nouveaux paramètres Antony et corrige les libellés des de…
8d4f4b7 
…scriptions
@jenovateurs @Shamzic
merge master et mise à jour de version OF / changelog
84bba5d
@jenovateurs @Shamzic
feat: ajout des references et mise à jour des valeurs de ressources
40ddbee
@jenovateurs @Shamzic
fix: mise à jour des seuils de la ville d'antony
c6678e9
@Shamzic Shamzic force-pushed the msa_ajout_aides_antony branch from 7e8941a to 99e72ff Compare November 6, 2024 12:58
Shamzic
Shamzic approved these changes Nov 6, 2024
View reviewed changes
Copy link
Contributor

@Shamzic Shamzic left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jenovateurs LGTM, bon pour toi ?

@Shamzic Shamzic force-pushed the msa_ajout_aides_antony branch from 6e55326 to 3edbd53 Compare November 6, 2024 13:19
@Shamzic Shamzic force-pushed the msa_ajout_aides_antony branch from 3edbd53 to f505688 Compare November 6, 2024 13:20
@jenovateurs jenovateurs merged commit db9b003 into master Nov 6, 2024
11 checks passed
@jenovateurs jenovateurs deleted the msa_ajout_aides_antony branch November 6, 2024 13:42
@jenovateurs jenovateurs mentioned this pull request Nov 7, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Kout95 Kout95 Kout95 requested changes

@Shamzic Shamzic Shamzic approved these changes

@guillett guillett guillett approved these changes

@mtifarine mtifarine Awaiting requested review from mtifarine

@sandcha sandcha Awaiting requested review from sandcha

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

Successfully merging this pull request may close these issues.
7 participants
@frtomas @Allan-CodeWorks @guillett @Shamzic @Kout95 @jenovateurs @mtifarine