search

sketch-city / harvey-api

Harvey Needs API
http://harveyneeds.org
GNU General Public License v3.0
23 stars 39 forks source link

Making Some Fields Required #70

Closed rmoscowitz closed 7 years ago

rmoscowitz commented 7 years ago

Also removing lat/long from forms (is this ok?).

Sorry @ironcoconut, this is another untested one. I'll set up a local environment if I do another PR, promise!

jwo commented 7 years ago

I think this means you cannot update the lat/lngs at all, ever. I think that's not the right move right now until we fix #28

@rmoscowitz I am OK with setting them as required fields but need you to confirm that existing fields where that data is not present, that you cannot update them without setting those values.

rmoscowitz commented 7 years ago

Ok, I'll leave the lat/longs alone. Also I'll go check for records missing my new required fields. Thanks for the feedback yall!