github
/
portainer
mirror of https://github.com/portainer/portainer
1
0
Fork 0
Code Issues Releases Wiki Activity

fix(notifications): limit only in header [EE-4815] (#8579) 

Co-authored-by: testa113 <testa113>
pull/8596/head
Ali 2023-03-03 15:08:15 +13:00 committed by GitHub
parent fd916bc8a2
commit 07df4b1591
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 7 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
app/react/components/PageHeader/NotificationsMenu.tsx
View File

@ -33,6 +33,7 @@ export function NotificationsMenu() {
notificationsStore,
(state) => state.userNotifications[user.Id]
);
const reducedNotifications = userNotifications?.slice(0, 50);
return (
<Menu>
@ -55,7 +56,7 @@ export function NotificationsMenu() {
<Icon icon={Bell} />
<span
className={
userNotifications?.length > 0 ? notificationStyles.badge : ''
reducedNotifications?.length > 0 ? notificationStyles.badge : ''
}
/>
</div>
@ -77,7 +78,7 @@ export function NotificationsMenu() {
<h4>Notifications</h4>
</div>
<div className={notificationStyles.itemLast}>
{userNotifications?.length > 0 && (
{reducedNotifications?.length > 0 && (
<Button
color="none"
onClick={(e) => {
@ -93,10 +94,10 @@ export function NotificationsMenu() {
</div>
</div>
</div>
{userNotifications?.length > 0 ? (
{reducedNotifications?.length > 0 ? (
<>
<div className={notificationStyles.notifications}>
{userNotifications.map((notification) => (
{reducedNotifications.map((notification) => (
<MenuLink
to="portainer.notifications"
params={{ id: notification.id }}

11
app/react/portainer/notifications/notifications-store.ts
View File

@ -20,22 +20,15 @@ export const notificationsStore = create<NotificationsState>()(
userNotifications: {},
addNotification: (userId: number, notification: ToastNotification) => {
set((state) => {
const currentUserNotifications =
state.userNotifications[userId] || [];
// keep the new notification at the start of the list, so sorting by newest time isn't required
const newUserNotifications = [
notification,
...currentUserNotifications,
...(state.userNotifications[userId] || []),
];
const maxNotifications = 50;
const reducedNotifications = newUserNotifications.slice(
0,
maxNotifications
);
return {
userNotifications: {
...state.userNotifications,
[userId]: reducedNotifications,
[userId]: newUserNotifications,
},
};
});