refactor: ignore invalid notifications

`lib/notifier/queries.js:getNotifications()` is only used in
`lib/new-admin/graphql/resolvers/notification.resolver.js`.

In the UI, the `valid` field is only used together with the `read`
field, with a true value.

lib/notifier/queries.js

@@ -54,7 +54,7 @@ const getValidNotifications = (type, detail) => {
 }
 
 const getNotifications = () => {
-  const sql = `SELECT * FROM notifications ORDER BY created DESC`
+  const sql = `SELECT * FROM notifications WHERE valid ORDER BY created DESC`
   return db.any(sql).catch(logger.error)
 }
 const setRead = (id, read) => {

new-lamassu-admin/src/components/NotificationCenter/NotificationCenter.js

@@ -25,7 +25,6 @@ const GET_NOTIFICATIONS = gql`
       message
       created
       read
-      valid
     }
     hasUnreadNotifications
     machines {
@@ -106,7 +105,6 @@ const NotificationCenter = ({
           deviceName={machines[n.detail.deviceId]}
           created={n.created}
           read={n.read}
-          valid={n.valid}
           toggleClear={() =>
             toggleClearNotification({
               variables: { id: n.id, read: !n.read }

new-lamassu-admin/src/components/NotificationCenter/NotificationRow.js

@@ -44,7 +44,6 @@ const NotificationRow = ({
   deviceName,
   created,
   read,
-  valid,
   toggleClear
 }) => {
   const classes = useStyles()
@@ -72,7 +71,7 @@ const NotificationRow = ({
     <div
       className={classnames(
         classes.notificationRow,
-        !read && valid ? classes.unread : ''
+        !read ? classes.unread : ''
       )}>
       <div className={classes.notificationRowIcon}>
         <div>{icon}</div>