codeforamerica/cfapi — pull request #73 by ondrae (2014-05-01 17:16:53+00:00)
Title: unicode names now work. Special characters in names will get that org skipped. With tests.

Hi @pui can you review this?

It removes all the urlquoting and encoding code, replaced by just not adding in any organizations that have those characters.

It also doesn't fail on those errors, just logs them in a new Error table, then continues. Engine light then checks for Errors in that table, and lets us know about them.

The tests are all there, but could probably use a few more comments.