-
Notifications
You must be signed in to change notification settings - Fork 462
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
uncaught TypeError: Object has no method 'toFixed' when using refreshmax #46
Comments
I suspect it has something to do with me adding decimals feature, I might have missed something. Will investigate. |
Hey James, can you send me example with issue? I am having trouble reproducing it... |
Thanks, you should be able to access this page - you can see the error when http://analytics-staging.thesocialbusiness.com/admin/the-social-business -James On Sat, Mar 23, 2013 at 6:04 AM, Bojan Djuricic [email protected]:
|
I get empty Facebook page, and an "Error" dialog... ? |
Sorry, the system depends on FB authentication. I thought I bypassed it for On Mon, Mar 25, 2013 at 8:19 AM, Bojan Djuricic [email protected]:
|
Can you give me just the part with gauge setup? Because we first need to determine if it's gauge setup related issue, and then look further if needed. |
I'm going to try to setup on jsfiddle. On Mon, Mar 25, 2013 at 8:29 AM, James Gentes
|
ok, just sent you a zip, not sure if it will run though.. On Mon, Mar 25, 2013 at 8:35 AM, Bojan Djuricic [email protected]:
|
It runs, I get two missing css errors, and DOM security exception 18 error, gauges run fine. I'm running it on Chrome 25. Tried on Firefox too, no DOM error, gauges run fine. |
Click on December / January a few times, that's what triggers the .refresh Thanks! -James On Tue, Mar 26, 2013 at 4:38 PM, Bojan Djuricic [email protected]:
|
+1, having this same issue |
Guys, please pull fresh update, and try. The problem is with type of value being sent to refresh method - it is supposed to be number. So technically it wasn't my fault :P I still have to check places in code where number is expected and do simple * 1 trick to make sure library is working with numbers, not strings. |
Yep, there was definitely an issue with James's example and new max value being sent as a string. This should work like charm now. Scream if I screwed up something :) |
I'm seeing this error crop up when I use the .refresh method: http://prntscr.com/x0kuo
Any ideas? I could be doing something very wrong on my end.
Thanks,
-James
The text was updated successfully, but these errors were encountered: