-
Notifications
You must be signed in to change notification settings - Fork 232
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
Latest code breaks all graphs #40
Comments
Trying this now.. |
Just placed the latest Can you provide some more info? such as what kind of errors you get (for example, from the javascript console). Thanks |
I see the same thing. building [Total Notifications] index.js:12 |
Reverting to acb8016 fixes the undefined |
Got it, might relate to a misbehaving pull request. I'll check it now and apply a fix soon. |
Problem was related to a condition where your Graphite source url does not have a query string. Hope this commit helps. |
@tomvachon @tempspace hope this fix helps? -thanks |
I pulled down the latest master this morning and all my graphs broke. After a bunch of individual file copying, it seems build/index.js is the culprit. Once I revert to my previous version, graphs work. Process is repeatable.
I will look more into this later...hoping that somebody else has already seen and fixed this to save me some time.
The text was updated successfully, but these errors were encountered: