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

Ie9 csv fix #1345

Closed
wants to merge 2 commits into from
Closed

Ie9 csv fix #1345

wants to merge 2 commits into from

Conversation

jfrazier
Copy link

Fix error parsing csv from xhr request callback object

Jason Frazier added 2 commits August 21, 2015 15:21
… param of xhr callback object. Copy responseText param to response param fixed the issue. Charts now render correctly.
@aendra-rininsland
Copy link
Member

aendra-rininsland commented May 2, 2016

Hi @jfrazier! Apologies for the incredibly long delay in getting to this PR!

LGTM, will happily merge it. First though —

  1. Please remove c3.js and c3.css (the built files) from your commits, these get built on releases.
  2. Squash your commits, and rebase them against master.

Thank you!

@aendra-rininsland aendra-rininsland added this to the 0.4.12 milestone May 2, 2016
@kt3k
Copy link
Member

kt3k commented Jun 8, 2017

Going to merge with appending comments. ( Approved by @masayuki0812 .

aendra-rininsland pushed a commit to aendra-rininsland/c3 that referenced this pull request Jun 10, 2017
* Fixes error parsing csv from xhr request callback object
* Fixes error in IE9 when parsing csv files. IE9 doesn't support response param of xhr callback object. Copy responseText param to response param fixed the issue. Charts now render correctly.
@aendra-rininsland
Copy link
Member

Merged in 6f3deac.

@kt3k
Copy link
Member

kt3k commented Jun 11, 2017

Published at v0.4.13.

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

Successfully merging this pull request may close these issues.

3 participants