-
Notifications
You must be signed in to change notification settings - Fork 19
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
base: master
Are you sure you want to change the base?
Conversation
@@ -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): |
There was a problem hiding this comment.
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?
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): |
There was a problem hiding this comment.
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.
…-aotearoa into kiwisaver
Opening on behalf of @JudyPuff after her team's work during the #BetterRules hackathon 🙂
These changes:
@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? 🙂