Skip to content

Commit

Permalink
fix: removed core type from ora app (#1040)
Browse files Browse the repository at this point in the history
  • Loading branch information
AhtishamShahid authored Apr 30, 2024
1 parent 2b82ac9 commit 35b5459
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
6 changes: 5 additions & 1 deletion src/notification-preferences/NotificationPreferenceRow.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,12 @@ const NotificationPreferenceRow = ({ appId, preferenceName }) => {
));
// eslint-disable-next-line react-hooks/exhaustive-deps
}, [appId, preferenceName]);

const tooltipId = `${preferenceName}-tooltip`;

if (appId === 'ora' && preferenceName === 'core') {
return null;
}

return (
<div className="d-flex mb-3" data-testid="notification-preference">
<div className="d-flex align-items-center mr-auto">
Expand Down
2 changes: 1 addition & 1 deletion src/notification-preferences/messages.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ const messages = defineMessages({
discussion {Discussions}
coursework {Course Work}
updates {Updates}
ora {ORA}
ora {Grading}
other {{key}}
}`,
description: 'Display text for Notification Types',
Expand Down

0 comments on commit 35b5459

Please sign in to comment.