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 support for gchart's newer library loader #113

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Add support for gchart's newer library loader #113

wants to merge 3 commits into from

Conversation

cmtjk
Copy link

@cmtjk cmtjk commented Mar 6, 2017

After encountering several problems loading Google Charts (gchart) via Ajax I've added support for Google Chart's preferred library loading method.

As of Feburary 16, 2016:

The version of Google Charts that remains available via the jsapi loader is no longer being updated consistently. The last update, for security purposes, was with a pre-release of v45.

Please use the new gstatic loader from now on [...]

Source

For compatibility reasons both methods are supported but you may decide to dismiss the old method in favor of the new one.

The following problems were solved with the proposed changes:

In addition using the new library loading method enables true asynchronous Ajax requests for loading charts whereas async: false must have been set using the old method.

To use the preferred method you should load the library like described in this example. This may be added to your documentation.

…thod.

For compatibility reasons graphos supports both methods (loader.js and older jsapi).
@cmtjk cmtjk changed the title Added support for gchart's newer library loader Add support for gchart's newer library loader Mar 6, 2017
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

Successfully merging this pull request may close these issues.

1 participant