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

Actualise un paramètre de l'ARS #1208

Merged
merged 4 commits into from
Nov 13, 2018
Merged

Actualise un paramètre de l'ARS #1208

merged 4 commits into from
Nov 13, 2018

Conversation

bfabre01
Copy link

  • Changement mineur.
  • Périodes concernées : toutes.
  • Détails :
    • Actualise un plafond de ressources de l'ARS

@Morendil
Copy link
Contributor

Merci @bfabre01 :)

Tu veux bien ajouter une référence législative sur le paramètre ? J'ai http://circulaires.legifrance.gouv.fr/pdf/2018/01/cir_42880.pdf - sauf si tu as mieux…

@bfabre01 bfabre01 force-pushed the actualise-plafond-ARS branch from be63507 to 51b164d Compare November 13, 2018 09:48
@bfabre01
Copy link
Author

@Morendil : merci pour la remarque. J'ai ajouté la référence. De préférence, il vaut mieux une référence avec le texte législatif ou règlementaire (ici, c'est un arrêté), ce que j'ai mis à la place de la circulaire (une circulaire étant un texte explicatif). Par contre, vu que la référence est propre au paramètre de 2018, je l'ai mise en commentaire à côté de la valeur. Est-ce que ça convient ?

@Morendil
Copy link
Contributor

Tu peux utiliser un champ reference qui est admis au même niveau que value, et plus utile car on peut l'exposer par l'API.

@bfabre01 bfabre01 merged commit 8ee3376 into master Nov 13, 2018
@bfabre01 bfabre01 deleted the actualise-plafond-ARS branch November 13, 2018 11:04
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