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.

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

Prevent incorrect eligibility to cheque energie in Saint Martin #949

Merged
merged 1 commit into from
Apr 3, 2018

Conversation

guillett
Copy link
Member

@guillett guillett commented Mar 16, 2018

  • Changement mineur.
  • Périodes concernées : À partir de 2018
  • Détails :
    • Suppression de l'éligibilité des foyers de Saint Martin au chèque énergie

@guillett guillett force-pushed the cheque_energie_st_martin branch 2 times, most recently from c0fafdd to cd09a79 Compare March 16, 2018 15:34
@guillett guillett requested a review from Anna-Livia March 16, 2018 16:23
@@ -33,20 +33,25 @@ class cheque_energie_eligibilite_logement(Variable):
reference = [
u"Article L124-1 du Code de l'énergie",
u"https://www.legifrance.gouv.fr/affichCodeArticle.do;jsessionid=5AB50D02153C9CB753729850314A2E17.tplgfr29s_1?idArticle=LEGIARTI000031057544&cidTexte=LEGITEXT000023983208&dateTexte=20180314",
u"Article LO6314-3 du Code général des collectivités territoriales",
Copy link
Contributor

Choose a reason for hiding this comment

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

🙌

@@ -130,3 +130,23 @@ class residence_mayotte(Variable):
def formula(menage, period, parameters):
depcom = menage('depcom', period)
return startswith(depcom, '976')


class residence_saint_bartelemy(Variable):
Copy link
Contributor

Choose a reason for hiding this comment

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

Cette variable ne semble pas être utilisée ... J'ai manqué quelque chose ?

Copy link
Member

Choose a reason for hiding this comment

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

Moi aussi je me demande où est-elle utilisée ?

@Anna-Livia Anna-Livia force-pushed the cheque_energie_st_martin branch from cd09a79 to c127843 Compare April 3, 2018 13:45
@Anna-Livia Anna-Livia merged commit 81591ee into master Apr 3, 2018
@bonjourmauko bonjourmauko deleted the cheque_energie_st_martin branch April 5, 2018 07:37
@bonjourmauko
Copy link
Member

@Anna-Livia Je n'arrive pas à comprendre pourquoi cette pull request à été mergé étant donné que la question que tu as posé reste sans réponse. Au vu de références législatives et du diff de cette pull request, me semble-t-il que celle-ci est inachevé.

@guillett Est-ce le cas ? Au vu de la LOI organique n° 2007-223 du 21 février 2007, sais-tu si l'article LO6314-3 du Code général des collectivités territoriales (Saint-Martin) peut être compris de la même façon que l'article LO6214-3 du Code général des collectivités territoriales (Saint-Barthélemy) (c'est-à-dire qu'aucune de deux collectivités n'est concernée par le chèque énergie) ?

@guillett
Copy link
Member Author

guillett commented Apr 5, 2018

@maukoquiroga des échanges en présentiel ont eu lieu. Je n'ai été informé que de l'inéligibilité des personnes vivants à Saint Martin.

Les personnes de Saint Barthélémy sont a-priori éligibles au chèque énergie.

Lors de l'ajout residence_saint_martin j'ai ajouté residence_saint_barthelemy pour de futurs usages.

@guillett
Copy link
Member Author

guillett commented Apr 5, 2018

@maukoquiroga comment es-tu arrivé ici, je suis surpris d'un commentaire sur une PR fermée depuis deux jours.

@bonjourmauko
Copy link
Member

Merci pour le follow up @guillett, as-tu d'autres éléments sur les échanges en présentiel à partager pour garder l'historique ?

comment es-tu arrivé ici

Github/Waffle

@guillett
Copy link
Member Author

guillett commented Apr 5, 2018

Oui, j'ai discuté avec @Anna-Livia.
J'ai dit qu'un request changes pour une variable non utilisée mais qui outille les futurs rédacteurs de formules c'est un peu dur.

En vacances pendant une semaine, ça n'a pas bougé et @Anna-Livia a mergé le PR récemment.

Pour information :
residence_mayotte teste startswith(depcom, '976').
residence_saint_barthelemy teste startswith(depcom, '977').
residence_saint_martin teste startswith(depcom, '978').

@bonjourmauko
Copy link
Member

Oui j'avais posé aussi la question notamment dû au fait que les deux sont des régies d'autre mer, et à faute d'info je ne savais pas si c'était un oublie ou pas.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants