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

Add housing.json file to situation examples; add to README #83

Merged
merged 4 commits into from
Feb 1, 2020
Merged

Add housing.json file to situation examples; add to README #83

merged 4 commits into from
Feb 1, 2020

Conversation

alexsoble
Copy link

  • Minor change -- Documentation and example JSON.
  • Details:
    • Add new housing example JSON to situation_examples folder. The housing example uses the Enum type; goal is to help newcomers to the OpenFisca world discover this.
    • Update README with copy-pasteable curl command for sending example JSON to local Web API.

Changes:

  • Change non-functional parts of this repository (for instance editing the README).

Thanks for considering this pull request!

I opened it after reading @sandcha's reply here — openfisca/openfisca-core#938 (comment) — which made me realize that I didn't fully understand all of the examples provided by this excellent template.

I know it's almost the weekend in Europe (and already the weekend in Asia-Pacific) — enjoy the weekend, no need to review this until next week or whenever there's capacity.

+ Goal -- make it easier for newcomers to the OpenFisca ecosystem to discover examples that use the Enum type within the country template
@benjello benjello merged commit 9264b43 into openfisca:master Feb 1, 2020
@alexsoble
Copy link
Author

🎉 @benjello @Morendil — Thank you for the reviews and approvals! My first PR into this repo! 🎉

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.

5 participants