-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Stable 12, user page, console warning #5115
Comments
Naive guess: related to #5114 (comment) |
Thanks for the hint - this resolves the missing message and the warning message in the console.
|
This is just a warning because some code (partly in apps) has not been adjusted to this change yet. Everything should work nevertheless. |
We are trying to get rid of all the Notification.hide() calls in the future. As of now you can safely ignore it, because the fallback code is still in place. It's basically a warning for us. As this is in the server code we will properly find it and replace it with the correct code. It's only helpful for developers of apps, that they know that there needs something improved on their side as well. Thanks for the report anyway 👍 |
Stable 12 (upgrade from 11.3 via upgrade app), loading users page.
js console says:
Missing argument $row in OC.Notification.hide() call, caller needs to be adjusted to only dismiss its own notification merged-template-prepend.js:2246:4
The text was updated successfully, but these errors were encountered: