Skip to content

Commit

Permalink
feat: added waffle flag for new notification view
Browse files Browse the repository at this point in the history
  • Loading branch information
ayesha-waris committed Oct 1, 2024
1 parent 093734b commit 9202028
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 3 deletions.
9 changes: 9 additions & 0 deletions openedx/core/djangoapps/notifications/config/waffle.py
Original file line number Diff line number Diff line change
Expand Up @@ -38,3 +38,12 @@
# .. toggle_target_removal_date: 2024-10-10
# .. toggle_tickets: INF-1304
ENABLE_ORA_GRADE_NOTIFICATION = CourseWaffleFlag(f"{WAFFLE_NAMESPACE}.enable_ora_grade_notifications", __name__)

# .. toggle_name: notifications.enable_new_notification_view
# .. toggle_implementation: WaffleFlag
# .. toggle_default: False
# .. toggle_description: Waffle flag to enable new notification view
# .. toggle_use_cases: temporary, open_edx
# .. toggle_creation_date: 2024-09-30
# .. toggle_tickets: INF-1603
ENABLE_NEW_NOTIFICATION_VIEW = CourseWaffleFlag(f"{WAFFLE_NAMESPACE}.enable_new_notification_view", __name__)
6 changes: 5 additions & 1 deletion openedx/core/djangoapps/notifications/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@

from common.djangoapps.student.models import CourseAccessRole, CourseEnrollment
from openedx.core.djangoapps.django_comment_common.models import Role
from openedx.core.djangoapps.notifications.config.waffle import ENABLE_NOTIFICATIONS
from openedx.core.djangoapps.notifications.config.waffle import ENABLE_NOTIFICATIONS, ENABLE_NEW_NOTIFICATION_VIEW
from openedx.core.lib.cache_utils import request_cached


Expand Down Expand Up @@ -47,6 +47,10 @@ def get_show_notifications_tray(user):
return show_notifications_tray


def get_is_new_notification_view_enabled():
return ENABLE_NEW_NOTIFICATION_VIEW.is_enabled()


def get_list_in_batches(input_list, batch_size):
"""
Divides the list of objects into list of list of objects each of length batch_size.
Expand Down
6 changes: 4 additions & 2 deletions openedx/core/djangoapps/notifications/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@
UserCourseNotificationPreferenceSerializer,
UserNotificationPreferenceUpdateSerializer,
)
from .utils import get_show_notifications_tray
from .utils import get_show_notifications_tray, get_is_new_notification_view_enabled


@allow_any_authenticated_user()
Expand Down Expand Up @@ -329,6 +329,7 @@ def get(self, request):
)
count_total = 0
show_notifications_tray = get_show_notifications_tray(self.request.user)
is_new_notification_view_enabled = get_is_new_notification_view_enabled()
count_by_app_name_dict = {
app_name: 0
for app_name in COURSE_NOTIFICATION_APPS
Expand All @@ -344,7 +345,8 @@ def get(self, request):
"show_notifications_tray": show_notifications_tray,
"count": count_total,
"count_by_app_name": count_by_app_name_dict,
"notification_expiry_days": settings.NOTIFICATIONS_EXPIRY
"notification_expiry_days": settings.NOTIFICATIONS_EXPIRY,
"is_new_notification_view_enabled": is_new_notification_view_enabled
})


Expand Down

0 comments on commit 9202028

Please sign in to comment.