@instructure/quiz-core 20.17.2-rc.9 → 20.17.3-rc.0

package/CHANGELOG.md

@@ -3,6 +3,20 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [20.17.2](https://gerrit.instructure.com/quizzes-ui/compare/v20.17.1...v20.17.2) (2024-09-05)
+
+
+### Bug Fixes
+
+* **a11y:** focus title/instructions when oqaat is on ([c49f940](https://gerrit.instructure.com/quizzes-ui/commits/c49f94049896a8fe1682a52cfde183944ea90be2))
+* **NQ:** add paragraph separator between paragraphs in NQ ([173dc07](https://gerrit.instructure.com/quizzes-ui/commits/173dc070d1fc73be0ebebe9b21ed2a2353784f01))
+* **NQ:** correct jumping behavior when a question is selected ([13780e4](https://gerrit.instructure.com/quizzes-ui/commits/13780e4ab7ecd7821da3219e5686eb0564710941))
+* **unit-tests:** Fix broken tests ([eaa945c](https://gerrit.instructure.com/quizzes-ui/commits/eaa945cdadd43c13296e09b1496171ec01f91ae4))
+
+
+
+
+
 ## [20.17.1](https://gerrit.instructure.com/quizzes-ui/compare/v20.17.0...v20.17.1) (2024-08-19)
 
 

package/es/banks/api/bankEntries.js

@@ -1,21 +1,21 @@
 import _toConsumableArray from "@babel/runtime/helpers/esm/toConsumableArray";
 import _slicedToArray from "@babel/runtime/helpers/esm/slicedToArray";
-import { clear } from "../../common/actions/ui.js";
-import Fetcher from "../../common/util/Fetcher.js";
-import PaginatedFetcher from "../../common/util/PaginatedFetcher.js";
-import { setBankFilters, associateEntries, disassociateEntry } from "../../common/actions/banks.js";
-import { handleBankEntryResponse, handleBankEntriesResponse } from "../../common/api/callHandlers.js";
-import { archiveBankEntry } from "../../common/actions/bankEntries.js";
-import { getBank } from "./banks.js";
-import { handleError } from "../../common/api/helpers.js";
+import { clear } from '../../common/actions/ui';
+import Fetcher from '../../common/util/Fetcher';
+import PaginatedFetcher from '../../common/util/PaginatedFetcher';
+import { setBankFilters, associateEntries, disassociateEntry } from '../../common/actions/banks';
+import { handleBankEntryResponse, handleBankEntriesResponse } from '../../common/api/callHandlers';
+import { archiveBankEntry } from '../../common/actions/bankEntries';
+import { getBank } from './banks';
+import { handleError } from '../../common/api/helpers';
 import get from 'lodash/get';
 import { BANKS_BANK_ENTRY_PAGINATION, GET_BANK_ENTRIES_CALL, GET_BANK_ENTRY_CALL, UPDATE_BANK_ENTRY_CALL, CREATE_BANK_ENTRY_CALL, REMOVE_BANK_ENTRY_CALL } from '@instructure/quiz-common';
 import t from '@instructure/quiz-i18n/es/format-message';
 export var getBankEntries = function getBankEntries(bankId, useBankSearch) {
   var params = arguments.length > 2 && arguments[2] !== void 0 ? arguments[2] : {};
   var page = params.page,
-      text = params.text,
-      filters = params.filters;
+    text = params.text,
+    filters = params.filters;
   var basePath = "/api/banks/".concat(bankId, "/bank_entries");
   var searchPath = "".concat(basePath, "/search");
   var interaction_type_ids = get(filters, 'interactionTypes', []);
@@ -32,7 +32,6 @@ export var getBankEntries = function getBankEntries(bankId, useBankSearch) {
         return e.id;
       });
     };
-
     var getRecordIds = useBankSearch ? getSearchIds : null;
     var fetcher = new PaginatedFetcher({
       callType: GET_BANK_ENTRIES_CALL,
@@ -42,21 +41,17 @@ export var getBankEntries = function getBankEntries(bankId, useBankSearch) {
     var path = useBankSearch ? searchPath : basePath;
     return fetcher.get(path, urlParams).then(function (r) {
       var _ref = useBankSearch ? [r[0].entries, r[1]] : r,
-          _ref2 = _slicedToArray(_ref, 2),
-          response = _ref2[0],
-          pageData = _ref2[1];
-
+        _ref2 = _slicedToArray(_ref, 2),
+        response = _ref2[0],
+        pageData = _ref2[1];
       var filterUpdate = [];
-
       if (useBankSearch) {
         var noInteractions = interaction_type_ids.length === 0;
         var noTags = tag_ids.length === 0;
-
         if (text === void 0 && noInteractions && noTags) {
           filterUpdate = [setBankFilters(bankId, r[0].filters)];
         }
       }
-
       dispatch([].concat(_toConsumableArray(filterUpdate), [handleBankEntriesResponse(response), associateEntries(bankId, response.map(function (be) {
         return be.id;
       }))]));
@@ -138,11 +133,9 @@ export var updateBankEntry = function updateBankEntry(bankId, bankEntryId, param
     };
     return fetcher.patch(url, updateBankEntryParams).then(function (response) {
       dispatch([handleBankEntryResponse(response), associateEntries(response.bank_id, [response.id])]);
-
       if (params.bankId && bankId !== params.bankId) {
         dispatch([clear("".concat(BANKS_BANK_ENTRY_PAGINATION, ":").concat(bankId)), clear("".concat(BANKS_BANK_ENTRY_PAGINATION, ":").concat(params.bankId)), getBank(bankId), getBank(params.bankId)]);
       }
-
       return response;
     });
   };

package/es/banks/api/banks.js

@@ -1,35 +1,32 @@
 import _objectSpread from "@babel/runtime/helpers/esm/objectSpread2";
 import _slicedToArray from "@babel/runtime/helpers/esm/slicedToArray";
-import Fetcher from "../../common/util/Fetcher.js";
-import PaginatedFetcher from "../../common/util/PaginatedFetcher.js";
-import ExhaustivePaginatedFetcher from "../../common/util/ExhaustivePaginatedFetcher.js";
-import { handleBanksResponse, handleBankResponse, handleShareBankResponse, handleShareBanksResponse } from "../../common/api/callHandlers.js";
-import { archiveBankEntry } from "../../common/actions/bankEntries.js";
-import { archiveBank } from "../../common/actions/banks.js";
-import { removeSharedBank, updateSharedBankPermission as updateSharedBankPermissionAction } from "../../common/actions/sharedBanks.js";
-import { clear, newActiveBankEntry, newActiveBank } from "../../common/actions/ui.js";
+import Fetcher from '../../common/util/Fetcher';
+import PaginatedFetcher from '../../common/util/PaginatedFetcher';
+import ExhaustivePaginatedFetcher from '../../common/util/ExhaustivePaginatedFetcher';
+import { handleBanksResponse, handleBankResponse, handleShareBankResponse, handleShareBanksResponse } from '../../common/api/callHandlers';
+import { archiveBankEntry } from '../../common/actions/bankEntries';
+import { archiveBank } from '../../common/actions/banks';
+import { removeSharedBank, updateSharedBankPermission as updateSharedBankPermissionAction } from '../../common/actions/sharedBanks';
+import { clear, newActiveBankEntry, newActiveBank } from '../../common/actions/ui';
 import t from '@instructure/quiz-i18n/es/format-message';
-import { handleError } from "../../common/api/helpers.js";
-import { removeFromPolling } from "./copyingBanksPoller.js";
+import { handleError } from '../../common/api/helpers';
+import { removeFromPolling } from './copyingBanksPoller';
 import { BANKS_BANK_PAGINATION, BANKS_BANK_ENTRY_PAGINATION, DELETE_BANK_CALL, UPDATE_BANK_CALL, CAN_EDIT, READ_ONLY, REMOVED_ACCESS, SHARED_WITH_USER, SHARED_WITH_COURSE, SHARED_WITH_ACCOUNT } from '@instructure/quiz-common';
 import { noop } from 'lodash';
 import { NIL } from 'uuid';
 var banksApiEndpoint = '/api/banks';
 var DEFAULT_PAGE_SIZE = 100;
-
 var _fetchBanks = function _fetchBanks(dispatch) {
   var params = arguments.length > 1 && arguments[1] !== void 0 ? arguments[1] : {};
   var fetcher = new PaginatedFetcher();
   return fetcher.get(banksApiEndpoint, params).then(function (_ref) {
     var _ref2 = _slicedToArray(_ref, 2),
-        response = _ref2[0],
-        pageData = _ref2[1];
-
+      response = _ref2[0],
+      pageData = _ref2[1];
     dispatch(handleBanksResponse(response));
     return pageData;
   });
 };
-
 export var getBanks = function getBanks() {
   var params = arguments.length > 0 && arguments[0] !== void 0 ? arguments[0] : {};
   var usedByCourseEnabled = arguments.length > 1 ? arguments[1] : void 0;
@@ -72,9 +69,9 @@ export var getBank = function getBank(bankId) {
     }).catch(function (err) {
       if (err.response.status === 404) {
         dispatch(clear(BANKS_BANK_PAGINATION));
-      } // Do nothing because this case is handled elsewhere.
+      }
+      // Do nothing because this case is handled elsewhere.
       // This catch is to suppress an console error for "unhandled in promise".
-
     });
   };
 };
@@ -106,16 +103,15 @@ export var createBank = function createBank(newBank, shareWithCourseUuid) {
       body: newBankData
     }).then(function (bankData) {
       dispatch([handleBankResponse(bankData), clear(BANKS_BANK_PAGINATION)]);
-
       if (shareWithCourseUuid) {
         simpleShareWithCourse(dispatch, bankData.id, shareWithCourseUuid);
       }
-
       return bankData;
     });
   };
-}; // not an action creator
+};
 
+// not an action creator
 var simpleShareWithCourse = function simpleShareWithCourse(dispatch, bankId, courseUuid) {
   var url = "".concat(banksApiEndpoint, "/").concat(bankId, "/shared_banks");
   var shareBankData = JSON.stringify({
@@ -133,7 +129,6 @@ var simpleShareWithCourse = function simpleShareWithCourse(dispatch, bankId, cou
     body: shareBankData
   });
 };
-
 export function getExistingBankAndEntries(bankId) {
   return function (dispatch, getState) {
     return dispatch([clear("".concat(BANKS_BANK_ENTRY_PAGINATION, ":").concat(bankId)), getBank(bankId)]);
@@ -157,11 +152,9 @@ export var updateBank = function updateBank(bank) {
       body: updatedBankData
     }).then(function (bankData) {
       var toDispatch = [handleBankResponse(bankData)];
-
       if (clearPagination) {
         toDispatch.push(clear(BANKS_BANK_PAGINATION));
       }
-
       dispatch(toDispatch);
       return bankData;
     });
@@ -177,13 +170,10 @@ export var deleteBank = function deleteBank(bankId) {
     });
     return fetcher.delete(url).then(function () {
       dispatch(clear(BANKS_BANK_PAGINATION));
-
       if (getState().getIn(['ui', 'activeBankId']) === bankId) {
         dispatch([newActiveBankEntry(void 0), newActiveBank(void 0)]);
       }
-
       var activeQuizId = getState().getIn(['quizzes', 'activeQuizId'], false);
-
       if (activeQuizId) {
         dispatch(archiveBank(bankId));
         getState().get('bankEntries', []).forEach(function (bankEntry) {
@@ -262,7 +252,6 @@ export var getSharedBank = function getSharedBank(bankId, entityType, entityId)
         };
         sharedBankInfo.push(placeholder);
       }
-
       dispatch(handleShareBanksResponse(sharedBankInfo));
     }).catch(function (err) {
       if (err.response.status === 401) {
@@ -288,9 +277,9 @@ export var getSharedBanks = function getSharedBanks(bank, getUserInfo, getCourse
           permission: CAN_EDIT,
           bank_id: bank.id
         });
-      } // USERS
-
+      }
 
+      // USERS
       var userSharedBanks = [];
       var userUuids = sharedBanksInfo.filter(function (sb) {
         return sb.entity_type === SHARED_WITH_USER;
@@ -302,7 +291,6 @@ export var getSharedBanks = function getSharedBanks(bank, getUserInfo, getCourse
           var sharedBankInfo = sharedBanksInfo.find(function (sb) {
             return sb.entity_type === SHARED_WITH_USER && sb.entity_id === userInfo.uuid;
           });
-
           if (sharedBankInfo) {
             userSharedBanks.push({
               id: sharedBankInfo.id,
@@ -316,8 +304,9 @@ export var getSharedBanks = function getSharedBanks(bank, getUserInfo, getCourse
           }
         });
         dispatch(handleShareBanksResponse(userSharedBanks));
-      }); // COURSES
+      });
 
+      // COURSES
       var courseSharedBanks = [];
       var courseUuids = sharedBanksInfo.filter(function (sb) {
         return sb.entity_type === SHARED_WITH_COURSE;
@@ -329,7 +318,6 @@ export var getSharedBanks = function getSharedBanks(bank, getUserInfo, getCourse
           var sharedBankInfo = sharedBanksInfo.find(function (sb) {
             return sb.entity_type === SHARED_WITH_COURSE && sb.entity_id === courseInfo.uuid;
           });
-
           if (sharedBankInfo) {
             courseSharedBanks.push({
               id: sharedBankInfo.id,
@@ -342,8 +330,9 @@ export var getSharedBanks = function getSharedBanks(bank, getUserInfo, getCourse
           }
         });
         dispatch(handleShareBanksResponse(courseSharedBanks));
-      }); // ACCOUNTS
+      });
 
+      // ACCOUNTS
       var accountSharedBanks = [];
       var accountUuids = sharedBanksInfo.filter(function (sb) {
         return sb.entity_type === SHARED_WITH_ACCOUNT;
@@ -355,7 +344,6 @@ export var getSharedBanks = function getSharedBanks(bank, getUserInfo, getCourse
           var sharedBankInfo = sharedBanksInfo.find(function (sb) {
             return sb.entity_type === SHARED_WITH_ACCOUNT && sb.entity_id === accountInfo.uuid;
           });
-
           if (sharedBankInfo) {
             accountSharedBanks.push({
               id: sharedBankInfo.id,

package/es/banks/api/copyingBanksPoller.js

@@ -1,8 +1,8 @@
-import _regeneratorRuntime from "@babel/runtime/regenerator";
+import _regeneratorRuntime from "@babel/runtime/helpers/esm/regeneratorRuntime";
 import _asyncToGenerator from "@babel/runtime/helpers/esm/asyncToGenerator";
 import t from '@instructure/quiz-i18n/es/format-message';
-import { addError, addSuccess } from "../../common/actions/alerts.js";
-import { getSpecificBanks } from "./banks.js";
+import { addError, addSuccess } from '../../common/actions/alerts';
+import { getSpecificBanks } from './banks';
 var pollingBanks = [];
 var currentlyPolling = false;
 export var addToPolling = function addToPolling(bankId, bankTitle) {
@@ -12,10 +12,9 @@ export var addToPolling = function addToPolling(bankId, bankTitle) {
       id: bankId,
       title: bankTitle
     });
-
     if (!currentlyPolling) {
       currentlyPolling = true;
-      handlePollingBanks(dispatch, getSpecificBanksFn);
+      _handlePollingBanks(dispatch, getSpecificBanksFn);
     }
   };
 };
@@ -24,74 +23,65 @@ export var removeFromPolling = function removeFromPolling(removeIds) {
     return !removeIds.includes(bank.id);
   });
 };
-
-var handlePollingBanks = /*#__PURE__*/function () {
-  var _ref = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee(dispatch) {
+var _handlePollingBanks = /*#__PURE__*/function () {
+  var _ref = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime().mark(function _callee(dispatch) {
     var getSpecificBanksFn,
-        updatedPollingBanks,
-        updatedPollingBankIds,
-        finishedCopying,
-        finishedCopyingIds,
-        _args = arguments;
-    return _regeneratorRuntime.wrap(function _callee$(_context) {
-      while (1) {
-        switch (_context.prev = _context.next) {
-          case 0:
-            getSpecificBanksFn = _args.length > 1 && _args[1] !== void 0 ? _args[1] : getSpecificBanks;
-            _context.next = 3;
-            return getSpecificBanksFn(pollingBanks.map(function (bank) {
-              return bank.id;
-            }))(dispatch);
-
-          case 3:
-            updatedPollingBanks = _context.sent;
-            updatedPollingBankIds = updatedPollingBanks.map(function (updatedBank) {
-              return updatedBank.id;
-            }); // banks that now have status 'waiting' have finished successfully
-            // banks that are no longer in the updated list but were in the previous list have
-            // failed to copy and therefore were archived
-
-            finishedCopying = updatedPollingBanks.filter(function (bank) {
-              return bank.status === 'waiting';
-            }).concat(pollingBanks.filter(function (prevBank) {
-              return !updatedPollingBankIds.includes(prevBank.id);
-            }));
-            finishedCopying.forEach(function (bank) {
-              var bankInfo = updatedPollingBanks.find(function (info) {
-                return info.id == bank.id;
-              });
-
-              if ((bankInfo === null || bankInfo === void 0 ? void 0 : bankInfo.status) === 'waiting') {
-                dispatch(addSuccess(t('Created "{name}" successfully! Duplicate can be seen in \'Shared with Me\' filter.', {
-                  name: bank.title
-                })));
-              } else if (!bankInfo) {
-                dispatch(addError(t('Failed to create "{name}". Please try again.', {
-                  name: bank.title
-                })));
-              }
+      updatedPollingBanks,
+      updatedPollingBankIds,
+      finishedCopying,
+      finishedCopyingIds,
+      _args = arguments;
+    return _regeneratorRuntime().wrap(function _callee$(_context) {
+      while (1) switch (_context.prev = _context.next) {
+        case 0:
+          getSpecificBanksFn = _args.length > 1 && _args[1] !== void 0 ? _args[1] : getSpecificBanks;
+          _context.next = 3;
+          return getSpecificBanksFn(pollingBanks.map(function (bank) {
+            return bank.id;
+          }))(dispatch);
+        case 3:
+          updatedPollingBanks = _context.sent;
+          updatedPollingBankIds = updatedPollingBanks.map(function (updatedBank) {
+            return updatedBank.id;
+          }); // banks that now have status 'waiting' have finished successfully
+          // banks that are no longer in the updated list but were in the previous list have
+          // failed to copy and therefore were archived
+          finishedCopying = updatedPollingBanks.filter(function (bank) {
+            return bank.status === 'waiting';
+          }).concat(pollingBanks.filter(function (prevBank) {
+            return !updatedPollingBankIds.includes(prevBank.id);
+          }));
+          finishedCopying.forEach(function (bank) {
+            var bankInfo = updatedPollingBanks.find(function (info) {
+              return info.id == bank.id;
             });
-            finishedCopyingIds = finishedCopying.map(function (bank) {
-              return bank.id;
-            });
-            removeFromPolling(finishedCopyingIds);
-
-            if (pollingBanks.length > 0) {
-              setTimeout(function () {
-                handlePollingBanks(dispatch);
-              }, 3000);
-            } else {
-              currentlyPolling = false;
+            if ((bankInfo === null || bankInfo === void 0 ? void 0 : bankInfo.status) === 'waiting') {
+              dispatch(addSuccess(t('Created "{name}" successfully! Duplicate can be seen in \'Shared with Me\' filter.', {
+                name: bank.title
+              })));
+            } else if (!bankInfo) {
+              dispatch(addError(t('Failed to create "{name}". Please try again.', {
+                name: bank.title
+              })));
             }
-
-          case 10:
-          case "end":
-            return _context.stop();
-        }
+          });
+          finishedCopyingIds = finishedCopying.map(function (bank) {
+            return bank.id;
+          });
+          removeFromPolling(finishedCopyingIds);
+          if (pollingBanks.length > 0) {
+            setTimeout(function () {
+              _handlePollingBanks(dispatch);
+            }, 3000);
+          } else {
+            currentlyPolling = false;
+          }
+        case 10:
+        case "end":
+          return _context.stop();
       }
     }, _callee);
   }));
-
   return function handlePollingBanks(_x) {
     return _ref.apply(this, arguments);
   };

package/es/banks/api/tagAssociations.js

@@ -1,8 +1,8 @@
-import _regeneratorRuntime from "@babel/runtime/regenerator";
+import _regeneratorRuntime from "@babel/runtime/helpers/esm/regeneratorRuntime";
 import _asyncToGenerator from "@babel/runtime/helpers/esm/asyncToGenerator";
-import Fetcher from "../../common/util/Fetcher.js";
-import { handleError } from "../../common/api/helpers.js";
-import { addTagAssociation, removePendingAdditionTagAssociation, removePendingDeletionTagAssociation, removeTagAssociation, setTagSuggestions } from "../../common/actions/tagAssociations.js";
+import Fetcher from '../../common/util/Fetcher';
+import { handleError } from '../../common/api/helpers';
+import { addTagAssociation, removePendingAdditionTagAssociation, removePendingDeletionTagAssociation, removeTagAssociation, setTagSuggestions } from '../../common/actions/tagAssociations';
 import { GET_TAG_SUGGESTIONS, CREATE_TAG_ASSOCIATION, DELETE_TAG_ASSOCIATION } from '@instructure/quiz-common';
 import t from '@instructure/quiz-i18n/es/format-message';
 export var getTagSuggestions = function getTagSuggestions(entryType, entryId, text) {
@@ -20,7 +20,6 @@ export var getTagSuggestions = function getTagSuggestions(entryType, entryId, te
     });
   };
 };
-
 var createTagAssociation = function createTagAssociation(dispatch, bankEntryId, entityType, entityId, tagValue) {
   var url = "/api/bank_entries/".concat(bankEntryId, "/tag_associations");
   var fetcher = new Fetcher({
@@ -35,36 +34,29 @@ var createTagAssociation = function createTagAssociation(dispatch, bankEntryId,
     return dispatch(addTagAssociation(entityType, entityId, tagAssociation));
   });
 };
-
 export var createTagAssociations = function createTagAssociations(bankEntryId, entityType, entityId, tagValues) {
   return function (dispatch, getState) {
     return Promise.all(tagValues.map( /*#__PURE__*/function () {
-      var _ref = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee(tagValue) {
-        return _regeneratorRuntime.wrap(function _callee$(_context) {
-          while (1) {
-            switch (_context.prev = _context.next) {
-              case 0:
-                _context.next = 2;
-                return createTagAssociation(dispatch, bankEntryId, entityType, entityId, tagValue);
-
-              case 2:
-                return _context.abrupt("return", dispatch(removePendingAdditionTagAssociation(entityType, entityId, tagValue)));
-
-              case 3:
-              case "end":
-                return _context.stop();
-            }
+      var _ref = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime().mark(function _callee(tagValue) {
+        return _regeneratorRuntime().wrap(function _callee$(_context) {
+          while (1) switch (_context.prev = _context.next) {
+            case 0:
+              _context.next = 2;
+              return createTagAssociation(dispatch, bankEntryId, entityType, entityId, tagValue);
+            case 2:
+              return _context.abrupt("return", dispatch(removePendingAdditionTagAssociation(entityType, entityId, tagValue)));
+            case 3:
+            case "end":
+              return _context.stop();
           }
         }, _callee);
       }));
-
       return function (_x) {
         return _ref.apply(this, arguments);
       };
     }()));
   };
 };
-
 var deleteTagAssociation = function deleteTagAssociation(dispatch, bankEntryId, entityType, entityId, tagAssociationId) {
   var url = "/api/bank_entries/".concat(bankEntryId, "/tag_associations/").concat(tagAssociationId);
   var fetcher = new Fetcher({
@@ -75,29 +67,23 @@ var deleteTagAssociation = function deleteTagAssociation(dispatch, bankEntryId,
     return dispatch([removeTagAssociation(entityType, entityId, tagAssociationId)]);
   });
 };
-
 export var deleteTagAssociations = function deleteTagAssociations(bankEntryId, entityType, entityId, tagAssociationIds) {
   return function (dispatch, getState) {
     return Promise.all(tagAssociationIds.map( /*#__PURE__*/function () {
-      var _ref2 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee2(tagAssociationId) {
-        return _regeneratorRuntime.wrap(function _callee2$(_context2) {
-          while (1) {
-            switch (_context2.prev = _context2.next) {
-              case 0:
-                _context2.next = 2;
-                return deleteTagAssociation(dispatch, bankEntryId, entityType, entityId, tagAssociationId);
-
-              case 2:
-                return _context2.abrupt("return", dispatch(removePendingDeletionTagAssociation(entityType, entityId, tagAssociationId)));
-
-              case 3:
-              case "end":
-                return _context2.stop();
-            }
+      var _ref2 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime().mark(function _callee2(tagAssociationId) {
+        return _regeneratorRuntime().wrap(function _callee2$(_context2) {
+          while (1) switch (_context2.prev = _context2.next) {
+            case 0:
+              _context2.next = 2;
+              return deleteTagAssociation(dispatch, bankEntryId, entityType, entityId, tagAssociationId);
+            case 2:
+              return _context2.abrupt("return", dispatch(removePendingDeletionTagAssociation(entityType, entityId, tagAssociationId)));
+            case 3:
+            case "end":
+              return _context2.stop();
           }
         }, _callee2);
       }));
-
       return function (_x2) {
         return _ref2.apply(this, arguments);
       };

package/es/banks/components/AddBankEntryModal/index.js

@@ -1,13 +1,12 @@
 import { bindActionCreators } from 'redux';
-import { connect } from "../../../common/react-redux.js";
-import { AddBankEntryModal as AddBankEntryModalPresenter } from "./presenter.js";
-import * as modalActions from "../../../common/actions/modal.js";
-import { createBankItem } from "../../../common/actions/modifications.js";
-import { interactionTypeFetch } from "../../../common/api/interactionTypes.js";
-import { getModalTrigger } from "../../../common/util/modalHelpers.js";
+import { connect } from '../../../common/react-redux';
+import { AddBankEntryModal as AddBankEntryModalPresenter } from './presenter';
+import * as modalActions from '../../../common/actions/modal';
+import { createBankItem } from '../../../common/actions/modifications';
+import { interactionTypeFetch } from '../../../common/api/interactionTypes';
+import { getModalTrigger } from '../../../common/util/modalHelpers';
 import { ADD_BANK_ENTRY_MODAL } from '@instructure/quiz-common';
-import { getInteractionTypes } from "../../../common/selectors/interactionTypes.js";
-
+import { getInteractionTypes } from '../../../common/selectors/interactionTypes';
 function mapStateToProps(state, props) {
   return {
     modalOpen: state.getIn(['modal', 'isOpen']) === ADD_BANK_ENTRY_MODAL,
@@ -15,7 +14,6 @@ function mapStateToProps(state, props) {
     interactionTypes: getInteractionTypes(state)
   };
 }
-
 var mapDispatchToProps = function mapDispatchToProps(dispatch) {
   return {
     closeModal: bindActionCreators(modalActions.closeModal, dispatch),
@@ -23,6 +21,5 @@ var mapDispatchToProps = function mapDispatchToProps(dispatch) {
     getInteractionTypes: bindActionCreators(interactionTypeFetch, dispatch)
   };
 };
-
 export var AddBankEntryModal = connect(mapStateToProps, mapDispatchToProps)(AddBankEntryModalPresenter);
 export default AddBankEntryModal;

package/es/banks/components/AddBankEntryModal/presenter.js

@@ -1,7 +1,7 @@
 import _classCallCheck from "@babel/runtime/helpers/esm/classCallCheck";
 import _createClass from "@babel/runtime/helpers/esm/createClass";
+import _callSuper from "@babel/runtime/helpers/esm/callSuper";
 import _inherits from "@babel/runtime/helpers/esm/inherits";
-import _createSuper from "@babel/runtime/helpers/esm/createSuper";
 import React, { Component } from 'react';
 import PropTypes from 'prop-types';
 import * as platform from 'platform';
@@ -10,24 +10,16 @@ import first from 'lodash/first';
 import { Button, CloseButton } from '@instructure/ui-buttons';
 import { Heading } from '@instructure/ui-heading';
 import t from '@instructure/quiz-i18n/es/format-message';
-import InteractionTypes from "../../../common/components/shared/InteractionTypes/index.js";
+import InteractionTypes from '../../../common/components/shared/InteractionTypes';
 import { XSMALL_SIDE_MARGIN, Modal, ModalHeader, ModalBody, ModalFooter } from '@instructure/quiz-common';
 export var AddBankEntryModal = /*#__PURE__*/function (_Component) {
-  _inherits(AddBankEntryModal, _Component);
-
-  var _super = _createSuper(AddBankEntryModal);
-
   function AddBankEntryModal() {
     var _this;
-
     _classCallCheck(this, AddBankEntryModal);
-
     for (var _len = arguments.length, args = new Array(_len), _key = 0; _key < _len; _key++) {
       args[_key] = arguments[_key];
     }
-
-    _this = _super.call.apply(_super, [this].concat(args));
-
+    _this = _callSuper(this, AddBankEntryModal, [].concat(args));
     _this.addItem = function (interactionTypeData) {
       var defaultInteractionData = interactionTypeData.getDefaultInteractionData();
       var scoringData = interactionTypeData.getDefaultScoringData(defaultInteractionData);
@@ -40,34 +32,25 @@ export var AddBankEntryModal = /*#__PURE__*/function (_Component) {
         scoringData: scoringData,
         userResponseType: first(interactionTypeData.userResponseTypeOptions)
       };
-
       _this.props.createBankItem(_this.props.bankId, 'Item', itemProps);
-
       _this.closeAction();
-
       _this.props.navigateToBankEntry('new', _this.props.bankId);
     };
-
     _this.addStimulus = function () {
       _this.props.createBankItem(_this.props.bankId, 'Stimulus');
-
       _this.closeAction();
-
       _this.props.navigateToBankEntry('new', _this.props.bankId);
     };
-
     _this.closeAction = function () {
       if (_this.props.modalTrigger) {
         _this.props.modalTrigger.focus();
       }
-
       _this.props.closeModal();
     };
-
     return _this;
   }
-
-  _createClass(AddBankEntryModal, [{
+  _inherits(AddBankEntryModal, _Component);
+  return _createClass(AddBankEntryModal, [{
     key: "componentDidMount",
     value: function componentDidMount() {
       this.props.getInteractionTypes();
@@ -109,9 +92,6 @@ export var AddBankEntryModal = /*#__PURE__*/function (_Component) {
       })), /*#__PURE__*/React.createElement(ModalFooter, null, this.cancelButton()));
     }
   }]);
-
-  AddBankEntryModal.displayName = "AddBankEntryModal";
-  return AddBankEntryModal;
 }(Component);
 AddBankEntryModal.propTypes = {
   modalOpen: PropTypes.bool.isRequired,