Skip to content

Commit 3696836

Browse files
authored
feat: save discussion alert dismissal (#1245)
1 parent 434fea3 commit 3696836

2 files changed

Lines changed: 10 additions & 1 deletion

File tree

src/course-outline/page-alerts/PageAlerts.jsx

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,8 +42,11 @@ const PageAlerts = ({
4242
const intl = useIntl();
4343
const dispatch = useDispatch();
4444
const studioBaseUrl = getConfig().STUDIO_BASE_URL;
45+
const discussionAlertDismissKey = `discussionAlertDismissed-${courseId}`;
4546
const [showConfigAlert, setShowConfigAlert] = useState(true);
46-
const [showDiscussionAlert, setShowDiscussionAlert] = useState(true);
47+
const [showDiscussionAlert, setShowDiscussionAlert] = useState(
48+
localStorage.getItem(discussionAlertDismissKey) === null,
49+
);
4750
const { newFiles, conflictingFiles, errorFiles } = useSelector(getPasteFileNotices);
4851

4952
const getAssetsUrl = () => {
@@ -84,6 +87,7 @@ const PageAlerts = ({
8487

8588
const onDismiss = () => {
8689
setShowDiscussionAlert(false);
90+
localStorage.setItem(discussionAlertDismissKey, 'true');
8791
};
8892

8993
return (

src/course-outline/page-alerts/PageAlerts.test.jsx

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -98,6 +98,11 @@ describe('<PageAlerts />', () => {
9898
expect(learnMoreBtn).toBeInTheDocument();
9999
expect(learnMoreBtn).toHaveAttribute('href', 'some-learn-more-url');
100100

101+
const dismissBtn = queryByText('Dismiss');
102+
await act(async () => fireEvent.click(dismissBtn));
103+
const discussionAlertDismissKey = `discussionAlertDismissed-${pageAlertsData.courseId}`;
104+
expect(localStorage.getItem(discussionAlertDismissKey)).toBe('true');
105+
101106
const feedbackLink = queryByText(messages.discussionNotificationFeedback.defaultMessage);
102107
expect(feedbackLink).toBeInTheDocument();
103108
expect(feedbackLink).toHaveAttribute('href', 'some-feedback-url');

0 commit comments

Comments
 (0)