Merge pull request #1058 from alleyinteractive/issue-1030

Prevent error by adding check before displaying notices

assets/js/pluginsidebar/sidebar.jsx

@@ -201,7 +201,14 @@ function Sidebar() {
 
   // Display notices whenever they change.
   useEffect(() => {
-    notices.forEach((notice) => displayNotification(notice.message, notice.type));
+    /* Adding a conditional here to prevent a sporadic error.
+    Leaving a console log in place in case we need to debug this further.
+    See: https://github.com/alleyinteractive/apple-news/issues/1030 */
+    if (Array.isArray(notices) && notices.length) {
+      notices.forEach((notice) => displayNotification(notice.message, notice.type));
+    } else {
+      console.log('Notices dispatched, but none to display.'); // eslint-disable-line no-console
+    }
   }, [displayNotification, notices]);
 
   return (