Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch to jsonlite? #31

Closed
eddelbuettel opened this issue Jan 27, 2017 · 3 comments
Closed

Switch to jsonlite? #31

eddelbuettel opened this issue Jan 27, 2017 · 3 comments

Comments

@eddelbuettel
Copy link
Owner

@eddelbuettel eddelbuettel commented Jan 27, 2017

Tried switching from (old) RJSONIO to (newer) jsonlite but simpleTests.R balks on file upload.

@restonslacker Can you take a peek? It is curl/jsonlite interaction. Somehow the token gets lost.

@restonslacker
Copy link
Contributor

@restonslacker restonslacker commented Jan 27, 2017

it appears that RJSONIO and jsonlite didn't quite align at how to unnest a single object. RSJONIO returned a vector with a single character and jsonlite was returning a list with a single string. that solved the issue for me.

@eddelbuettel
Copy link
Owner Author

@eddelbuettel eddelbuettel commented Jan 27, 2017

I looked at it, suspecting such a minor difference, but somewhat I was too dense. Or had too little coffee.

@restonslacker
Copy link
Contributor

@restonslacker restonslacker commented Jan 27, 2017

given that it took me 5 commits to make that change, i'd say you're not alone.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.