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

Ajoute une référence à l'ASS #1357

Merged
merged 1 commit into from
Jul 20, 2019
Merged

Ajoute une référence à l'ASS #1357

merged 1 commit into from
Jul 20, 2019

Conversation

guillett
Copy link
Member

  • Changement mineur
  • Détails :
    • Ajoute une référence à l'ASS

Copy link
Member

@bonjourmauko bonjourmauko left a comment

Choose a reason for hiding this comment

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

Hello @guillett, merci ! Quelques suggestions non bloquantes.

@@ -10,6 +10,10 @@ class ass_precondition_remplie(Variable):
entity = Individu
label = u"Éligible à l'ASS"
definition_period = MONTH
reference = [
u"Article R5423-1 du Code du travail",
Copy link
Member

Choose a reason for hiding this comment

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

In py3, strings are unicode by default.

Suggested change
u"Article R5423-1 du Code du travail",
"Article R5423-1 du Code du travail",

@@ -10,6 +10,10 @@ class ass_precondition_remplie(Variable):
entity = Individu
label = u"Éligible à l'ASS"
definition_period = MONTH
reference = [
u"Article R5423-1 du Code du travail",
u"https://www.legifrance.gouv.fr/affichCodeArticle.do?cidTexte=LEGITEXT000006072050&idArticle=LEGIARTI000018525084&dateTexte=20190618&categorieLien=cid#LEGIARTI000018525084"
Copy link
Member

Choose a reason for hiding this comment

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

In py3, strings are unicode by default.

Suggested change
u"https://www.legifrance.gouv.fr/affichCodeArticle.do?cidTexte=LEGITEXT000006072050&idArticle=LEGIARTI000018525084&dateTexte=20190618&categorieLien=cid#LEGIARTI000018525084"
"https://www.legifrance.gouv.fr/affichCodeArticle.do?cidTexte=LEGITEXT000006072050&idArticle=LEGIARTI000018525084&dateTexte=20190618&categorieLien=cid#LEGIARTI000018525084"

@bonjourmauko bonjourmauko merged commit fc613bb into master Jul 20, 2019
@bonjourmauko bonjourmauko deleted the ass-doc branch July 20, 2019 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants