Merge pull request #1143 from 18F/js-clear-notifications

Clear notifications upon changing pages
cloud-gov · Jun 28, 2017 · 25f8098 · 25f8098
2 parents ed4c63f + b7a9185
commit 25f8098
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/static_src/components/users.jsx b/static_src/components/users.jsx
@@ -153,6 +153,9 @@ export default class Users extends React.Component {
           status="finish"
         />
       );
+    } else {
+      // If there's nothing, let's reset the notification to null.
+      notification = null;
     }
 
     if (this.state.currentType === ORG_NAME) {

diff --git a/static_src/stores/user_store.js b/static_src/stores/user_store.js
@@ -8,7 +8,7 @@ import Immutable from 'immutable';
 
 import BaseStore from './base_store.js';
 import cfApi from '../util/cf_api.js';
-import { userActionTypes } from '../constants.js';
+import { userActionTypes, errorActionTypes } from '../constants.js';
 
 export class UserStore extends BaseStore {
   constructor() {
@@ -282,6 +282,16 @@ export class UserStore extends BaseStore {
         break;
       }
 
+      case errorActionTypes.CLEAR: {
+        this._error = null;
+        this._saving = false;
+        this._inviteNotification = {};
+        this._inviteError = null;
+        this._loading = {};
+        this.emitChange();
+        break;
+      }
+
       default:
         break;
     }