tmaybe
commented
8 years ago I reviewed and also:
- am handling unescaped query parameters, with test, closing #285
- standardized the way we query for properties
- fixed a breaking test
- fixed some non-breaking unicode errors in tests
@ondrae this is ready for your review/merge
On the Brigade Project Search page, we want to show Brigade and Code for All project, but not Government projects, by default. Currently the CfAPI didn't allow you to do that. This PR changes that.
We want the default results from /projects to be the same as if you searched for http://www.codeforamerica.org/brigade/projects?type=Brigade,Code+for+All. This PR will make both of those work, at least on the CfAPI side.
Hey @tmaybe I could use your review of this. I think it could also use your work for changing how we are doing string matching, to a safer way that would prevent sql injection.