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

[WIP] Add Home Grant and Kiwisaver #17

Open
wants to merge 23 commits into
base: master
Choose a base branch
from
Open

[WIP] Add Home Grant and Kiwisaver #17

wants to merge 23 commits into from

Conversation

MattiSG
Copy link
Member

@MattiSG MattiSG commented May 21, 2018

Opening on behalf of @JudyPuff after her team's work during the #BetterRules hackathon 🙂

  • Tax and benefit system evolution.
  • Impacted periods: all.
  • Impacted areas: Income, KiwiSaver, Citizenship, Home Grant,
  • Details:
    • Adds eligibility and amount calculation for Home Grant.

These changes:

  • Add a new feature (for instance adding a variable)

@verbman it looks like this branch was based on some of your previous work on other branches, it would make it much easier to review if it was rebased and only the Home Grant / Kiwisaver-specific stuff was listed. Could you please open PRs for your other branches so we can review them as well? 🙂

@MattiSG MattiSG changed the title Add Kiwisaver Add Home Grant and Kiwisaver May 21, 2018
@MattiSG
Copy link
Member Author

MattiSG commented May 21, 2018

My preferred course of action would be:

  1. Open PRs for @verbman's dependent branches.
  2. Rebase @JudyPuff's work.
  3. Have @JudyPuff open a PR against master 🙂

@@ -29,23 +29,99 @@ class ks_contrib_duration_satisfied(Variable):
def formula(persons, period):
return persons('ks_contrib_duration', period) >= 3

class homestart_grant(Variable):
class homestart_grant__kiwisaver(Variable):
Copy link
Member

Choose a reason for hiding this comment

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

What does the double underscore indicate?

@Br3nda
Copy link
Member

Br3nda commented May 29, 2018

I've added some codestyle/linter fixes -- we can squash those commits when we merge

label = u"Years the prospective home buyer has been contributing continuously to their kiwisaver account"


class contrib_duration__kiwisaver(Variable):
Copy link
Member

Choose a reason for hiding this comment

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

This class name is used twice in this file.

@Br3nda Br3nda changed the title Add Home Grant and Kiwisaver [WIP] Add Home Grant and Kiwisaver Aug 12, 2018
@ghost ghost assigned Br3nda Nov 10, 2018
@mermop mermop removed this from Inbox in OpenFisca Aotearoa Jun 22, 2020
verbman pushed a commit that referenced this pull request May 7, 2024
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.

None yet

3 participants