Notifications could previously come from hidden course instances, which made it impossible to clear the notification icon due to CourseVisiblePermission.
This is now fixed by filtering courses that are not visible to students and invalidating the CachedNotifications when a course instance is made hidden.
Description
What?
Notifications could previously come from hidden course instances, which made it impossible to clear the notification icon due to
CourseVisiblePermission
.This is now fixed by filtering courses that are not visible to students and invalidating the
CachedNotifications
when a course instance is made hidden.Fixes #683
Testing
What type of test did you run?
Tested that a student no longer sees notifications from hidden courses.
Did you test the changes in
Translation
Programming style
Have you updated the README or other relevant documentation?
Is it Done?