-
Notifications
You must be signed in to change notification settings - Fork 14
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
ProgressButton not defined, because there is no update #61
Comments
Thanks for reminding me. I've seen that. It's harmless, but should be fixed. I'm thinking of just putting the same button in the "up-to-date" message but disabled and with display:none, but haven't had time to try it. |
Hi Bob, yeah it is harmless, but I (and probably all others, including you) don't like JS errors in the console :) Why not adapt the JS and not even initialize the button, if it's not there? |
I agree completely and intend to fix it. It's just that I'm in the middle of about six projects right now, some of which are not running at all. ;) |
Okay, thanks for the info :) |
When just visiting the dashboard on a up-to-date MODX,
I get the following JS-error in the console:
The JS is right, there is no Button:
The text was updated successfully, but these errors were encountered: