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

Problem when loading remote data with stock charts #5

Open
markedmondson opened this issue May 25, 2015 · 3 comments
Open

Problem when loading remote data with stock charts #5

markedmondson opened this issue May 25, 2015 · 3 comments

Comments

@markedmondson
Copy link

With data with a date format "YYYY-MM-DD" and no dateFormat set within periodSelector, I'm getting this error, although the graph renders as expected:
Uncaught TypeError: Cannot read property 'setDefaultPeriod' of undefined

Fiddle: http://jsfiddle.net/6g6tcbb1/1/

When the dateFormat is set, the graph size is shrunk and no longer visible: http://jsfiddle.net/6g6tcbb1/2/

I'm guessing the periodSelector isn't waiting until the data has been fully loaded but it's hard to dig through the minified amcharts files.

@martynasma
Copy link
Collaborator

Thanks for bringing it to our attention. Grab the latest commit from the repo for a fixed version.

@markedmondson
Copy link
Author

Thanks for the prompt fix but I think it still exists with the latest version: http://jsfiddle.net/6g6tcbb1/2/ when the dateFormat is set unless I'm missing something?

@martynasma
Copy link
Collaborator

Hi Mark,

I can't seem to be able to reproduce it. However, I can see there's another issue with the chart you are using as a test. If the periodSelector does not have any pre-defined periods, the panel is hidden. This is a bug in JavaScript Stock Chart and is not related to Data Loader. We'll fix it.

In the interim, if you set some pre-defined periods in your chart, you should be good to go.

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