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

Preliminary fix for displaying raw text/html #2263

Merged
merged 1 commit into from Apr 28, 2016

Conversation

Projects
None yet
2 participants
@dannon
Copy link
Member

commented Apr 28, 2016

Raw unicode return here was being dropped by the webapp or proxy. Encoding the output here is appropriate, but as a TODO I'm going to try to push this logic up into the framework so inside the webapp it's all unicode all the time.

This is a preliminary fix for displaying raw text/html unicodify'd da…
…ta (html reports, etc).

TODO: Push this logic up into the framework so inside the webapp it's
all unicode all the time.
@dannon

This comment has been minimized.

Copy link
Member Author

commented Apr 28, 2016

fixes #2240 (and once this is actually confirmed to resolve the issue on test.galaxyproject.org I'll backport to 16.04 where this is also an issue)

@nturaga nturaga merged commit bf0491f into galaxyproject:dev Apr 28, 2016

4 checks passed

api test Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
framework test Build finished.
Details
toolshed test Build finished.
Details
@dannon

This comment has been minimized.

Copy link
Member Author

commented Apr 28, 2016

Thanks @nturaga!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.