Skip to content
This repository was archived by the owner on Jun 18, 2024. It is now read-only.

Add HHS's validator to the tools list #197

Merged
merged 1 commit into from
Nov 26, 2013
Merged

Add HHS's validator to the tools list #197

merged 1 commit into from
Nov 26, 2013

Conversation

JoshData
Copy link
Contributor

Adding my validator, per suggestion in #150.

@waldoj
Copy link

waldoj commented Nov 23, 2013

👍 It's great to have another validator—thanks, Josh!

@gbinal
Copy link
Contributor

gbinal commented Nov 23, 2013

+1

@MarinaNitze
Copy link
Contributor

+1 Woot!

@haleyvandyck
Copy link
Contributor

Lovely to have healthy competition! Thanks a ton for this @JoshData, very helpful.

haleyvandyck pushed a commit that referenced this pull request Nov 26, 2013
Add HHS's validator to the tools list
@haleyvandyck haleyvandyck merged commit 2a7def8 into project-open-data:master Nov 26, 2013
@cew821
Copy link

cew821 commented Nov 26, 2013

@JoshData Is your validator on Github? I'd love to take a look. For some reason it's not able to parse our JSON.

@JoshData
Copy link
Contributor Author

@cew821 : It is: https://github.com/HHS/ckanext-datajson/blob/master/ckanext/datajson/datajsonvalidator.py.

I'd be glad to also take a look at the problem if you send me the URL. Or post an issue over on that repo. :)

@philipashlock
Copy link
Contributor

My guess is it's a character encoding issue, eg invalid newlines. See http://forum.jquery.com/topic/json-with-newlines-in-strings-should-be-valid#14737000000866332 and http://stackoverflow.com/posts/17846592/revisions

Using jsonlint I get "invalid token" on that json file. See http://stackoverflow.com/questions/6679244/json-invalid-token

@cew821
Copy link

cew821 commented Nov 27, 2013

Thanks for the help all. It wound up being a BOM character at the beginning of the file (probably from Microsoft notepad or Excel). I edited it and now I'm getting to use @JoshData 's awesome validator! Seems like this one is by far the most polished, great job!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants