discourse
/
discourse-assign
mirror of https://github.com/discourse/discourse-assign.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

FIX: use grouped_unread_notifications (#373) 

Use `grouped_unread_notifications` instead of `grouped_unread_high_priority_notifications`

Related PR: https://github.com/discourse/discourse/pull/18132
This commit is contained in:
Krzysztof Kotlarek 2022-08-31 11:16:13 +10:00 committed by GitHub
parent 030d349aa0
commit 0d586bb93c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
assets/javascripts/discourse/components/user-menu/assigns-list.js
View File

@ -71,7 +71,7 @@ export default class UserMenuAssignNotificationsList extends UserMenuNotificatio
}
get _unreadAssignedNotificationsCount() {
const key = `grouped_unread_high_priority_notifications.${this.site.notification_types.assigned}`;
const key = `grouped_unread_notifications.${this.site.notification_types.assigned}`;
// we're retrieving the value with get() so that Ember tracks the property
// and re-renders the UI when it changes.
// we can stop using `get()` when the User model is refactored into native

4
test/javascripts/acceptance/assigns-tab-user-menu-test.js
View File

@ -234,7 +234,7 @@ acceptance("Discourse Assign | experimental user menu", function (needs) {
needs.user({
redesigned_user_menu_enabled: true,
can_assign: true,
grouped_unread_high_priority_notifications: {
grouped_unread_notifications: {
34: 173, // assigned notification type
},
});
@ -286,7 +286,7 @@ acceptance("Discourse Assign | experimental user menu", function (needs) {
);
updateCurrentUser({
grouped_unread_high_priority_notifications: {},
grouped_unread_notifications: {},
});
assert.notOk(