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

Add pretty printing as an optional paramater to the JSON Response #23

Closed
lewismc opened this issue Apr 13, 2015 · 2 comments
Closed

Add pretty printing as an optional paramater to the JSON Response #23

lewismc opened this issue Apr 13, 2015 · 2 comments

Comments

@lewismc
Copy link
Owner

lewismc commented Apr 13, 2015

Right now we get a JSON response object and there is no option to pretty print it so it is difficult to interpret at a glance.
We should consider implementing something like
https://clojuredocs.org/clojure.pprint
as a mechanism for improving aesthetics and interpretation.

@cartermp
Copy link
Collaborator

I took a look at this a while back and again this morning, and it seems to be difficult to do. Running the data set through pprint would actually just insert a ton of tabs and spaces which would result in horrendous output because ring interprets those as literal characters. I can't seem to find a middleware support pretty-printing at this time.

@lewismc
Copy link
Owner Author

lewismc commented Apr 29, 2015

I also had a look and found it hellish to implement. This has to do with support for this kind of thing within Clojure I think. I am +1 for leaving it and closing it off.

@lewismc lewismc closed this as completed Apr 29, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants