fix: bad prop causing bulk mgmt errors to not show (#184)

On success and set uploadSuccess to true every time. This is important for unit testing.
[EDUCATOR-5796]

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@edx/frontend-app-gradebook",
-  "version": "1.4.26",
+  "version": "1.4.28",
   "lockfileVersion": 1,
   "requires": true,
   "dependencies": {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@edx/frontend-app-gradebook",
-  "version": "1.4.26",
+  "version": "1.4.28",
   "description": "edx editable gradebook-ui to manipulate grade overrides on subsections",
   "repository": {
     "type": "git",

src/components/Gradebook/BulkManagement.jsx

@@ -91,7 +91,7 @@ export class BulkManagement extends React.Component {
           <StatusAlert
             alertType="danger"
             dialog={this.props.bulkImportError}
-            isOpen={this.props.bulkImportError}
+            open={!!this.props.bulkImportError}
             dismissible={false}
           />
           <StatusAlert

src/data/reducers/grades.js

@@ -136,12 +136,16 @@ const grades = (state = initialState, action) => {
         bulkManagement: rest,
       };
     }
-    case UPLOAD_COMPLETE:
+    case UPLOAD_COMPLETE: {
       return {
         ...state,
         showSpinner: false,
-        bulkManagement: { uploadSuccess: true, ...state.bulkManagement },
+        bulkManagement: {
+          ...state.bulkManagement,
+          uploadSuccess: true,
+        },
       };
+    }
     case UPLOAD_ERR:
       return {
         ...state,