#162 : Notifications - Remove the completed challenges section from the popup

Author: luizrrodrigues

dist/dev/index.js

@@ -48,7 +48,6 @@ export default class NotificationList extends React.Component {
   constructor (props) {
     super(props)
     this.state = {
-      completedSection: [],
       nonCompletedSection: [],
       unreadCount: 0
     }
@@ -65,8 +64,7 @@ export default class NotificationList extends React.Component {
 
   render () {
     const { onClose, onSettings, notifications, onDismiss, unReadNotifications,
-      markNotificationAsRead, markAllNotificationAsRead,
-      dismissChallengeNotifications, auth } = this.props
+      markNotificationAsRead, markAllNotificationAsRead, auth } = this.props
 
     return (
       <>
@@ -126,7 +124,7 @@ export default class NotificationList extends React.Component {
           </div>
         </div>
         <div className={styles['noti-body']}>
-          <Fragment key='nonComplete'>
+          <Fragment>
             {
               this.challenges(_.uniq((notifications || [])).filter(t => !eventTypes.PROJECT.COMPLETED.includes(t.eventType))).map((challenge, challengeIdx) =>
                 (
@@ -146,23 +144,6 @@ export default class NotificationList extends React.Component {
                 ))
             }
           </Fragment>
-          <div className={styles['completed-header']}>Completed Challenges</div>
-          <Fragment key='completed'>
-            {
-              this.challenges(_.uniq((notifications || [])).filter(t => eventTypes.PROJECT.COMPLETED.includes(t.eventType))).map((challenge, challengeIdx) =>
-                (
-                  <div key={`noti-completed-${challengeIdx}`} className={cn([styles['challenge-title'], styles['completed-challenge']])}>
-                    <span>{challenge.challengeTitle}</span>
-                    <div className={styles['dismiss-challenge']} onClick={c => {
-                      const challegeId = challenge && challenge.items && challenge.items.length && challenge.items[0].sourceId
-                      if (challegeId) {
-                        dismissChallengeNotifications(challegeId, auth.tokenV3)
-                      }
-                    }}>&times;</div>
-                  </div>
-                ))
-            }
-          </Fragment>
         </div>
         <div className={styles['view-all-notifications']}>
           <a href='/notifications'>View all Notifications</a>
@@ -211,6 +192,5 @@ NotificationList.propTypes = {
   onClose: PropTypes.func,
   unReadNotifications: PropTypes.bool,
   markNotificationAsRead: PropTypes.func.isRequired,
-  markAllNotificationAsRead: PropTypes.func.isRequired,
-  dismissChallengeNotifications: PropTypes.func.isRequired
+  markAllNotificationAsRead: PropTypes.func.isRequired
 }

dist/prod/index.js

@@ -175,38 +175,7 @@
       padding-left: 20px;
       margin-top: -2px;
       padding: 5px 20px;
-    }
-    .completed-challenge {
-      display: flex;
-      justify-content: space-between;
-      background-color: $gray-04;
-      font-size: 14px;
-      @include Roboto-Bold;
-      font-weight: 500;
-      line-height: 22px;
-      padding: 5px 20px;
-      margin-top: 5px;
-      .dismiss-challenge {
-        cursor:pointer;
-        color: #aaaaaa;
-        @include Roboto-Regular;
-        font-size: 20px;
-        margin: auto 0px;
-      }
-    }
-    .completed-header {
-        background-color: $gray-06;
-        text-transform: uppercase;
-        color: $gray-90;
-        font-size: 14px;
-        @include Roboto-Bold;
-        line-height: 22px;
-        padding-top: 15px;
-        padding-bottom: 13px;
-        padding-left: 20px;
-        page-break-after: auto;
-    }
-    
+    }    
     .greybar-section {
       &:last-child {
         .items {