Skip to content

Commit

Permalink
frontend: Notifications: Disable Notifications
Browse files Browse the repository at this point in the history
There's a number of problems with Notifications. With some clusters
the number of warnings makes it not useful. Because they keep showing
new notifications over and over again.

The idea is to disable it for now, and revisit it again once we can
prove that it is working well.

Signed-off-by: René Dudfield <renedudfield@microsoft.com>
  • Loading branch information
illume committed Mar 26, 2024
1 parent 24f2322 commit 0e79b49
Showing 1 changed file with 2 additions and 7 deletions.
9 changes: 2 additions & 7 deletions frontend/src/components/App/TopBar.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,6 @@ import { drawerWidth } from '../Sidebar';
import HeadlampButton from '../Sidebar/HeadlampButton';
import { setWhetherSidebarOpen } from '../Sidebar/sidebarSlice';
import { AppLogo } from './AppLogo';
import Notifications from './Notifications';

export interface TopBarProps {}

Expand Down Expand Up @@ -328,11 +327,7 @@ export function PureTopBar({
...appBarActions,
{
id: DefaultAppBarAction.NOTIFICATION,
action: (
<MenuItem onClick={handleMenuClose}>
<Notifications />
</MenuItem>
),
action: null,
},
{
id: DefaultAppBarAction.SETTINGS,
Expand Down Expand Up @@ -391,7 +386,7 @@ export function PureTopBar({
...appBarActions,
{
id: DefaultAppBarAction.NOTIFICATION,
action: <Notifications />,
action: null,
},
{
id: DefaultAppBarAction.SETTINGS,
Expand Down

0 comments on commit 0e79b49

Please sign in to comment.