search

18F / dsnap_rules

Other
3 stars 2 forks source link

validate.py does not require "state_or_territory" #1

Closed edmullen closed 6 years ago

edmullen commented 6 years ago

In new_rules/new_rules/validate.py, isn't "state_or_territory" necessary to correctly determine income eligibility?

I see "is_authorized_representative" is also not required, which seems correct.

vrajmohan commented 6 years ago

Good point. I was vacillating on the state_or_territory being an attribute of the request vs configuration and didn't commit to one way or another.

vrajmohan commented 6 years ago

We'll eliminate this field from the request and look it up in the backend using the DR# - see #11.