@instructure/quiz-core 20.36.1-rc.4 → 20.36.1-rc.6

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

@@ -19,7 +19,6 @@ function mapStateToProps(state, props) {
   return {
     itemBankSearchTableEnabled: featureOn('item_bank_search_table'),
     usedByCourseEnabled: featureOn('filter_item_banks_used_by_course'),
-    subAccountBankSharingEnabled: featureOn('sub_account_bank_sharing'),
     scopeType: scopeType,
     sortBy: sortBy,
     sortOrder: sortOrder,

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

@@ -222,7 +222,7 @@ export var Banks = (_dec = withStyleOverrides(generateStyle, generateComponentTh
   }, {
     key: "topLevelBankFilterName",
     value: function topLevelBankFilterName() {
-      return this.props.subAccountBankSharingEnabled ? t('Shared with Me') : t('All Banks');
+      return t('Shared with Me');
     }
   }, {
     key: "renderFilters",
@@ -235,7 +235,7 @@ export var Banks = (_dec = withStyleOverrides(generateStyle, generateComponentTh
         translation: t('Institution Banks')
       }, {
         slug: 'courses',
-        translation: this.props.subAccountBankSharingEnabled ? t('Shared with Courses') : t('Banks Shared to Course')
+        translation: t('Shared with Courses')
       }];
       if (this.props.scope.uuid) {
         scopeOptions.push({
@@ -243,7 +243,7 @@ export var Banks = (_dec = withStyleOverrides(generateStyle, generateComponentTh
           translation: t('This Course')
         });
       }
-      if (this.props.subAccountBankSharingEnabled && this.isSubAccountLaunch()) {
+      if (this.isSubAccountLaunch()) {
         // Put this option below "Insitution Banks"
         scopeOptions.splice(2, 0, {
           slug: 'this_account',
@@ -302,7 +302,7 @@ export var Banks = (_dec = withStyleOverrides(generateStyle, generateComponentTh
         onChange: this.handleSearchInputChange,
         placeholder: t('Search Banks'),
         "data-automation": "sdk-search-bank-textinput"
-      })), this.props.subAccountBankSharingEnabled && !this.hideAccountSearch() && jsx(Grid.Col, {
+      })), !this.hideAccountSearch() && jsx(Grid.Col, {
         width: 4
       }, jsx(AsyncSearch, {
         getShareableSubaccounts: this.props.getShareableSubaccounts,
@@ -488,7 +488,6 @@ export var Banks = (_dec = withStyleOverrides(generateStyle, generateComponentTh
   usedByCourseEnabled: PropTypes.bool,
   itemBankSearchTableEnabled: PropTypes.bool,
   setUI: PropTypes.func.isRequired,
-  subAccountBankSharingEnabled: PropTypes.bool,
   iceTopNavBarEnabled: PropTypes.bool,
   styles: PropTypes.object,
   rootAccountUuid: PropTypes.string
@@ -515,7 +514,6 @@ export var Banks = (_dec = withStyleOverrides(generateStyle, generateComponentTh
   searchSelection: [],
   usedByCourseEnabled: false,
   itemBankSearchTableEnabled: false,
-  subAccountBankSharingEnabled: false,
   iceTopNavBarEnabled: false,
   rootAccountUuid: null
 }, _Banks.contextTypes = {

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

@@ -4,14 +4,12 @@ import { connect } from "../../../common/react-redux.js";
 import { shareBank, getSharedBanks, updateSharedBankPermission, getBank, updateBank } from "../../api/banks.js";
 import { removeSharedBank } from "../../../common/actions/sharedBanks.js";
 import { withConfirm } from "../../../common/actions/modal.js";
-import { featureOn } from "../../../common/util/featureCheck.js";
 import SharingModal from "./presenter.js";
 function mapStateToProps(state, ownProps) {
   var sharedBanks = state.get('sharedBanks', Map()).filter(function (sharedBank) {
     return sharedBank.get('bankId') === ownProps.bank.id;
   }).toList();
   return {
-    subAccountBankSharingEnabled: featureOn('sub_account_bank_sharing'),
     sharedBanks: sharedBanks
   };
 }

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

@@ -1,6 +1,6 @@
+import _defineProperty from "@babel/runtime/helpers/esm/defineProperty";
 import _regeneratorRuntime from "@babel/runtime/regenerator";
 import _asyncToGenerator from "@babel/runtime/helpers/esm/asyncToGenerator";
-import _defineProperty from "@babel/runtime/helpers/esm/defineProperty";
 import _slicedToArray from "@babel/runtime/helpers/esm/slicedToArray";
 import _objectSpread from "@babel/runtime/helpers/esm/objectSpread2";
 import _classCallCheck from "@babel/runtime/helpers/esm/classCallCheck";
@@ -176,27 +176,7 @@ export var SharingModalPresenter = (_dec = withStyleOverrides(generateStyle, gen
       return originalEntity.get('permission', CAN_EDIT);
     };
     _this.handleEntityRemove = function (entity, undoRemove) {
-      if (_this.props.subAccountBankSharingEnabled) {
-        _this.confirmPermissionChange(entity, REMOVED_ACCESS);
-      } else {
-        var permission = undoRemove ? _this.originalPermissionForEntity(entity) : REMOVED_ACCESS;
-        _this.setState(function (prevState) {
-          var sharesIndex = prevState.shares.findIndex(function (e) {
-            return e.get('id') == entity.get('id');
-          });
-          var newEntitiesIndex = prevState.newEntities.findIndex(function (e) {
-            return e.get('id') == entity.get('id');
-          });
-          var removedNewEntity = newEntitiesIndex > -1;
-          var newShares = removedNewEntity ? prevState.shares.delete(sharesIndex) : prevState.shares.setIn([sharesIndex, 'permission'], permission);
-          var newEntities = removedNewEntity ? prevState.newEntities.delete(newEntitiesIndex) : prevState.newEntities;
-          return {
-            shares: newShares,
-            newEntities: newEntities,
-            entitiesWithChangedPermission: Object.assign({}, prevState.entitiesWithChangedPermission, _defineProperty({}, entity.get('id'), permission))
-          };
-        }, _this.updateEntitiesCount);
-      }
+      _this.confirmPermissionChange(entity, REMOVED_ACCESS);
     };
     _this.handleSearchPermissionChange = function (e, _ref4) {
       var value = _ref4.value;
@@ -223,21 +203,13 @@ export var SharingModalPresenter = (_dec = withStyleOverrides(generateStyle, gen
           permission: _this.state.searchPermission
         });
       }));
-      if (_this.props.subAccountBankSharingEnabled) {
-        newEntities.forEach(function (entity) {
-          _this.props.onShare(entity.toJS());
-          _this.props.shareBank(_this.sharedBankForEntity(entity), _this.props.getUserInfo);
-        });
-        _this.setState({
-          searchSelection: []
-        });
-      } else {
-        _this.setState({
-          searchSelection: [],
-          shares: newEntities.concat(_this.state.shares),
-          newEntities: newEntities.concat(_this.state.newEntities)
-        }, _this.updateEntitiesCount);
-      }
+      newEntities.forEach(function (entity) {
+        _this.props.onShare(entity.toJS());
+        _this.props.shareBank(_this.sharedBankForEntity(entity), _this.props.getUserInfo);
+      });
+      _this.setState({
+        searchSelection: []
+      });
     };
     _this.updatePermission = /*#__PURE__*/function () {
       var _ref5 = _asyncToGenerator(/*#__PURE__*/_regeneratorRuntime.mark(function _callee(entity, permission) {
@@ -282,19 +254,7 @@ export var SharingModalPresenter = (_dec = withStyleOverrides(generateStyle, gen
       });
     };
     _this.handleEntityPermissionChange = function (entity, permission) {
-      if (_this.props.subAccountBankSharingEnabled) {
-        _this.confirmPermissionChange(entity, permission);
-      } else {
-        _this.setState(function (prevState) {
-          var index = prevState.shares.findIndex(function (share) {
-            return share.get('id') === entity.get('id');
-          });
-          return {
-            shares: prevState.shares.setIn([index, 'permission'], permission),
-            entitiesWithChangedPermission: Object.assign({}, prevState.entitiesWithChangedPermission, _defineProperty({}, entity.get('id'), permission))
-          };
-        });
-      }
+      _this.confirmPermissionChange(entity, permission);
     };
     _this.toggleAccountShare = function (event) {
       _this.setState(function (prevState) {
@@ -383,7 +343,7 @@ export var SharingModalPresenter = (_dec = withStyleOverrides(generateStyle, gen
   }, {
     key: "componentDidUpdate",
     value: function componentDidUpdate(prevProps, prevState) {
-      if (this.props.subAccountBankSharingEnabled && this.state.accountPermission !== prevState.accountPermission && this.state.accountPermission !== this.props.bank.accountPermission) {
+      if (this.state.accountPermission !== prevState.accountPermission && this.state.accountPermission !== this.props.bank.accountPermission) {
         var updatedBank = this.props.bank.set('accountPermission', this.state.accountPermission);
         this.props.updateBank(updatedBank, false);
       }
@@ -502,37 +462,20 @@ export var SharingModalPresenter = (_dec = withStyleOverrides(generateStyle, gen
       var existingAccounts = existingShares.filter(function (s) {
         return s.get('entityType') === SHARED_WITH_ACCOUNT;
       });
-      if (this.props.subAccountBankSharingEnabled) {
-        return this.renderSubaccountSharingEnabledEntities(existingPeople, existingCourses, existingAccounts);
-      }
-      return this.renderOriginalDesignEntities(existingPeople, existingCourses);
+      return this.renderSubaccountSharingEnabledEntities(existingPeople, existingCourses, existingAccounts);
     }
   }, {
     key: "renderFooter",
     value: function renderFooter() {
-      if (this.props.subAccountBankSharingEnabled) {
-        return jsx(Button, {
-          color: "primary",
-          margin: "0 x-small 0 0",
-          onClick: this.handleDismiss
-        }, t('Close'));
-      }
-      return jsx(React.Fragment, null, jsx(Button, {
-        "data-automation": "sdk-shared-bank-cancel-button",
-        color: "primary-inverse",
-        onClick: this.handleDismiss,
-        margin: "0 x-small 0 0"
-      }, t('Cancel')), jsx(Button, {
-        "data-automation": "sdk-shared-bank-done-button",
+      return jsx(Button, {
         color: "primary",
-        onClick: this.handleSubmit,
-        margin: "0 x-small 0 0"
-      }, t('Done')));
+        margin: "0 x-small 0 0",
+        onClick: this.handleDismiss
+      }, t('Close'));
     }
   }, {
     key: "render",
     value: function render() {
-      var subAccountBankSharingEnabled = this.props.subAccountBankSharingEnabled;
       return jsx(Modal, {
         open: this.props.isOpen,
         onDismiss: this.handleDismiss,
@@ -545,7 +488,7 @@ export var SharingModalPresenter = (_dec = withStyleOverrides(generateStyle, gen
         placement: "end",
         offset: "medium",
         screenReaderLabel: t('Close')
-      })), jsx(ModalBody, null, !subAccountBankSharingEnabled && this.renderRootAccountSharing(), !subAccountBankSharingEnabled && this.renderSearchSection(), this.state.shares.size >= 0 ? this.renderEntitiesSection() : _ref7), jsx(ModalFooter, null, this.renderFooter()));
+      })), jsx(ModalBody, null, this.state.shares.size >= 0 ? this.renderEntitiesSection() : _ref7), jsx(ModalFooter, null, this.renderFooter()));
     }
   }]);
   SharingModalPresenter.displayName = "SharingModalPresenter";
@@ -580,7 +523,6 @@ export var SharingModalPresenter = (_dec = withStyleOverrides(generateStyle, gen
   removeSharedBank: PropTypes.func.isRequired,
   shareBank: PropTypes.func.isRequired,
   sharedBanks: ImmutablePropTypes.list,
-  subAccountBankSharingEnabled: PropTypes.bool,
   updateBank: PropTypes.func.isRequired,
   updateSharedBankPermission: PropTypes.func.isRequired,
   updateSharedCount: PropTypes.func,
@@ -595,7 +537,6 @@ export var SharingModalPresenter = (_dec = withStyleOverrides(generateStyle, gen
   onDismiss: noop,
   onShare: function onShare() {},
   sharedBanks: List(),
-  subAccountBankSharingEnabled: false,
   updateSharedCount: Function.prototype
 }, _SharingModalPresenter)) || _class);
 export default SharingModalPresenter;

package/es/common/components/resources/item/ItemEdit/presenter.js

@@ -15,7 +15,6 @@ import t from '@instructure/quiz-i18n/es/format-message';
 import { propsForInteractionEdit } from "../../../../util/interactionTypePropsHelper.js";
 import elementsForSelectors from "../../../../../common/util/ElementsForSelectors.js";
 import { toErrors } from "../../../../util/instUIMessages.js";
-import { featureOn } from "../../../../../common/util/featureCheck.js";
 export var ItemEdit = /*#__PURE__*/function (_Component) {
   _inherits(ItemEdit, _Component);
   var _super = _createSuper(ItemEdit);
@@ -63,7 +62,7 @@ export var ItemEdit = /*#__PURE__*/function (_Component) {
       var launchContextUuid = this.props.scope.launch_context_uuid;
       var launchContextTitle = this.props.scope.title;
       // If editing within a quiz, there is no account launch context
-      if (featureOn('sub_account_bank_sharing') && rootAccountName) {
+      if (rootAccountName) {
         launchContexts.push({
           uuid: this.props.rootAccountUuid,
           name: rootAccountName

package/lib/banks/components/Banks/index.js

@@ -26,7 +26,6 @@ function mapStateToProps(state, props) {
   return {
     itemBankSearchTableEnabled: (0, _featureCheck.featureOn)('item_bank_search_table'),
     usedByCourseEnabled: (0, _featureCheck.featureOn)('filter_item_banks_used_by_course'),
-    subAccountBankSharingEnabled: (0, _featureCheck.featureOn)('sub_account_bank_sharing'),
     scopeType: scopeType,
     sortBy: sortBy,
     sortOrder: sortOrder,

package/lib/banks/components/Banks/presenter.js

@@ -230,7 +230,7 @@ var Banks = (_dec = (0, _quizCommon.withStyleOverrides)(_styles.default, _theme.
   }, {
     key: "topLevelBankFilterName",
     value: function topLevelBankFilterName() {
-      return this.props.subAccountBankSharingEnabled ? (0, _formatMessage.default)('Shared with Me') : (0, _formatMessage.default)('All Banks');
+      return (0, _formatMessage.default)('Shared with Me');
     }
   }, {
     key: "renderFilters",
@@ -243,7 +243,7 @@ var Banks = (_dec = (0, _quizCommon.withStyleOverrides)(_styles.default, _theme.
         translation: (0, _formatMessage.default)('Institution Banks')
       }, {
         slug: 'courses',
-        translation: this.props.subAccountBankSharingEnabled ? (0, _formatMessage.default)('Shared with Courses') : (0, _formatMessage.default)('Banks Shared to Course')
+        translation: (0, _formatMessage.default)('Shared with Courses')
       }];
       if (this.props.scope.uuid) {
         scopeOptions.push({
@@ -251,7 +251,7 @@ var Banks = (_dec = (0, _quizCommon.withStyleOverrides)(_styles.default, _theme.
           translation: (0, _formatMessage.default)('This Course')
         });
       }
-      if (this.props.subAccountBankSharingEnabled && this.isSubAccountLaunch()) {
+      if (this.isSubAccountLaunch()) {
         // Put this option below "Insitution Banks"
         scopeOptions.splice(2, 0, {
           slug: 'this_account',
@@ -310,7 +310,7 @@ var Banks = (_dec = (0, _quizCommon.withStyleOverrides)(_styles.default, _theme.
         onChange: this.handleSearchInputChange,
         placeholder: (0, _formatMessage.default)('Search Banks'),
         "data-automation": "sdk-search-bank-textinput"
-      })), this.props.subAccountBankSharingEnabled && !this.hideAccountSearch() && (0, _emotion.jsx)(_uiGrid.Grid.Col, {
+      })), !this.hideAccountSearch() && (0, _emotion.jsx)(_uiGrid.Grid.Col, {
         width: 4
       }, (0, _emotion.jsx)(_index7.default, {
         getShareableSubaccounts: this.props.getShareableSubaccounts,
@@ -496,7 +496,6 @@ var Banks = (_dec = (0, _quizCommon.withStyleOverrides)(_styles.default, _theme.
   usedByCourseEnabled: _propTypes.default.bool,
   itemBankSearchTableEnabled: _propTypes.default.bool,
   setUI: _propTypes.default.func.isRequired,
-  subAccountBankSharingEnabled: _propTypes.default.bool,
   iceTopNavBarEnabled: _propTypes.default.bool,
   styles: _propTypes.default.object,
   rootAccountUuid: _propTypes.default.string
@@ -523,7 +522,6 @@ var Banks = (_dec = (0, _quizCommon.withStyleOverrides)(_styles.default, _theme.
   searchSelection: [],
   usedByCourseEnabled: false,
   itemBankSearchTableEnabled: false,
-  subAccountBankSharingEnabled: false,
   iceTopNavBarEnabled: false,
   rootAccountUuid: null
 }, _Banks.contextTypes = {

package/lib/banks/components/SharingModal/index.js

@@ -11,14 +11,12 @@ var _reactRedux = require("../../../common/react-redux.js");
 var _banks = require("../../api/banks.js");
 var _sharedBanks = require("../../../common/actions/sharedBanks.js");
 var _modal = require("../../../common/actions/modal.js");
-var _featureCheck = require("../../../common/util/featureCheck.js");
 var _presenter = _interopRequireDefault(require("./presenter.js"));
 function mapStateToProps(state, ownProps) {
   var sharedBanks = state.get('sharedBanks', (0, _immutable.Map)()).filter(function (sharedBank) {
     return sharedBank.get('bankId') === ownProps.bank.id;
   }).toList();
   return {
-    subAccountBankSharingEnabled: (0, _featureCheck.featureOn)('sub_account_bank_sharing'),
     sharedBanks: sharedBanks
   };
 }

package/lib/banks/components/SharingModal/presenter.js

@@ -6,9 +6,9 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.default = exports.SharingModalPresenter = void 0;
+var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 var _regenerator = _interopRequireDefault(require("@babel/runtime/regenerator"));
 var _asyncToGenerator2 = _interopRequireDefault(require("@babel/runtime/helpers/asyncToGenerator"));
-var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 var _slicedToArray2 = _interopRequireDefault(require("@babel/runtime/helpers/slicedToArray"));
 var _objectSpread2 = _interopRequireDefault(require("@babel/runtime/helpers/objectSpread2"));
 var _classCallCheck2 = _interopRequireDefault(require("@babel/runtime/helpers/classCallCheck"));
@@ -184,27 +184,7 @@ var SharingModalPresenter = (_dec = (0, _quizCommon.withStyleOverrides)(_styles.
       return originalEntity.get('permission', _quizCommon.CAN_EDIT);
     };
     _this.handleEntityRemove = function (entity, undoRemove) {
-      if (_this.props.subAccountBankSharingEnabled) {
-        _this.confirmPermissionChange(entity, _quizCommon.REMOVED_ACCESS);
-      } else {
-        var permission = undoRemove ? _this.originalPermissionForEntity(entity) : _quizCommon.REMOVED_ACCESS;
-        _this.setState(function (prevState) {
-          var sharesIndex = prevState.shares.findIndex(function (e) {
-            return e.get('id') == entity.get('id');
-          });
-          var newEntitiesIndex = prevState.newEntities.findIndex(function (e) {
-            return e.get('id') == entity.get('id');
-          });
-          var removedNewEntity = newEntitiesIndex > -1;
-          var newShares = removedNewEntity ? prevState.shares.delete(sharesIndex) : prevState.shares.setIn([sharesIndex, 'permission'], permission);
-          var newEntities = removedNewEntity ? prevState.newEntities.delete(newEntitiesIndex) : prevState.newEntities;
-          return {
-            shares: newShares,
-            newEntities: newEntities,
-            entitiesWithChangedPermission: Object.assign({}, prevState.entitiesWithChangedPermission, (0, _defineProperty2.default)({}, entity.get('id'), permission))
-          };
-        }, _this.updateEntitiesCount);
-      }
+      _this.confirmPermissionChange(entity, _quizCommon.REMOVED_ACCESS);
     };
     _this.handleSearchPermissionChange = function (e, _ref4) {
       var value = _ref4.value;
@@ -231,21 +211,13 @@ var SharingModalPresenter = (_dec = (0, _quizCommon.withStyleOverrides)(_styles.
           permission: _this.state.searchPermission
         });
       }));
-      if (_this.props.subAccountBankSharingEnabled) {
-        newEntities.forEach(function (entity) {
-          _this.props.onShare(entity.toJS());
-          _this.props.shareBank(_this.sharedBankForEntity(entity), _this.props.getUserInfo);
-        });
-        _this.setState({
-          searchSelection: []
-        });
-      } else {
-        _this.setState({
-          searchSelection: [],
-          shares: newEntities.concat(_this.state.shares),
-          newEntities: newEntities.concat(_this.state.newEntities)
-        }, _this.updateEntitiesCount);
-      }
+      newEntities.forEach(function (entity) {
+        _this.props.onShare(entity.toJS());
+        _this.props.shareBank(_this.sharedBankForEntity(entity), _this.props.getUserInfo);
+      });
+      _this.setState({
+        searchSelection: []
+      });
     };
     _this.updatePermission = /*#__PURE__*/function () {
       var _ref5 = (0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee(entity, permission) {
@@ -290,19 +262,7 @@ var SharingModalPresenter = (_dec = (0, _quizCommon.withStyleOverrides)(_styles.
       });
     };
     _this.handleEntityPermissionChange = function (entity, permission) {
-      if (_this.props.subAccountBankSharingEnabled) {
-        _this.confirmPermissionChange(entity, permission);
-      } else {
-        _this.setState(function (prevState) {
-          var index = prevState.shares.findIndex(function (share) {
-            return share.get('id') === entity.get('id');
-          });
-          return {
-            shares: prevState.shares.setIn([index, 'permission'], permission),
-            entitiesWithChangedPermission: Object.assign({}, prevState.entitiesWithChangedPermission, (0, _defineProperty2.default)({}, entity.get('id'), permission))
-          };
-        });
-      }
+      _this.confirmPermissionChange(entity, permission);
     };
     _this.toggleAccountShare = function (event) {
       _this.setState(function (prevState) {
@@ -391,7 +351,7 @@ var SharingModalPresenter = (_dec = (0, _quizCommon.withStyleOverrides)(_styles.
   }, {
     key: "componentDidUpdate",
     value: function componentDidUpdate(prevProps, prevState) {
-      if (this.props.subAccountBankSharingEnabled && this.state.accountPermission !== prevState.accountPermission && this.state.accountPermission !== this.props.bank.accountPermission) {
+      if (this.state.accountPermission !== prevState.accountPermission && this.state.accountPermission !== this.props.bank.accountPermission) {
         var updatedBank = this.props.bank.set('accountPermission', this.state.accountPermission);
         this.props.updateBank(updatedBank, false);
       }
@@ -510,37 +470,20 @@ var SharingModalPresenter = (_dec = (0, _quizCommon.withStyleOverrides)(_styles.
       var existingAccounts = existingShares.filter(function (s) {
         return s.get('entityType') === _quizCommon.SHARED_WITH_ACCOUNT;
       });
-      if (this.props.subAccountBankSharingEnabled) {
-        return this.renderSubaccountSharingEnabledEntities(existingPeople, existingCourses, existingAccounts);
-      }
-      return this.renderOriginalDesignEntities(existingPeople, existingCourses);
+      return this.renderSubaccountSharingEnabledEntities(existingPeople, existingCourses, existingAccounts);
     }
   }, {
     key: "renderFooter",
     value: function renderFooter() {
-      if (this.props.subAccountBankSharingEnabled) {
-        return (0, _emotion.jsx)(_uiButtons.Button, {
-          color: "primary",
-          margin: "0 x-small 0 0",
-          onClick: this.handleDismiss
-        }, (0, _formatMessage.default)('Close'));
-      }
-      return (0, _emotion.jsx)(_react.default.Fragment, null, (0, _emotion.jsx)(_uiButtons.Button, {
-        "data-automation": "sdk-shared-bank-cancel-button",
-        color: "primary-inverse",
-        onClick: this.handleDismiss,
-        margin: "0 x-small 0 0"
-      }, (0, _formatMessage.default)('Cancel')), (0, _emotion.jsx)(_uiButtons.Button, {
-        "data-automation": "sdk-shared-bank-done-button",
+      return (0, _emotion.jsx)(_uiButtons.Button, {
         color: "primary",
-        onClick: this.handleSubmit,
-        margin: "0 x-small 0 0"
-      }, (0, _formatMessage.default)('Done')));
+        margin: "0 x-small 0 0",
+        onClick: this.handleDismiss
+      }, (0, _formatMessage.default)('Close'));
     }
   }, {
     key: "render",
     value: function render() {
-      var subAccountBankSharingEnabled = this.props.subAccountBankSharingEnabled;
       return (0, _emotion.jsx)(_quizCommon.Modal, {
         open: this.props.isOpen,
         onDismiss: this.handleDismiss,
@@ -553,7 +496,7 @@ var SharingModalPresenter = (_dec = (0, _quizCommon.withStyleOverrides)(_styles.
         placement: "end",
         offset: "medium",
         screenReaderLabel: (0, _formatMessage.default)('Close')
-      })), (0, _emotion.jsx)(_quizCommon.ModalBody, null, !subAccountBankSharingEnabled && this.renderRootAccountSharing(), !subAccountBankSharingEnabled && this.renderSearchSection(), this.state.shares.size >= 0 ? this.renderEntitiesSection() : _ref7), (0, _emotion.jsx)(_quizCommon.ModalFooter, null, this.renderFooter()));
+      })), (0, _emotion.jsx)(_quizCommon.ModalBody, null, this.state.shares.size >= 0 ? this.renderEntitiesSection() : _ref7), (0, _emotion.jsx)(_quizCommon.ModalFooter, null, this.renderFooter()));
     }
   }]);
   SharingModalPresenter.displayName = "SharingModalPresenter";
@@ -588,7 +531,6 @@ var SharingModalPresenter = (_dec = (0, _quizCommon.withStyleOverrides)(_styles.
   removeSharedBank: _propTypes.default.func.isRequired,
   shareBank: _propTypes.default.func.isRequired,
   sharedBanks: _reactImmutableProptypes.default.list,
-  subAccountBankSharingEnabled: _propTypes.default.bool,
   updateBank: _propTypes.default.func.isRequired,
   updateSharedBankPermission: _propTypes.default.func.isRequired,
   updateSharedCount: _propTypes.default.func,
@@ -603,7 +545,6 @@ var SharingModalPresenter = (_dec = (0, _quizCommon.withStyleOverrides)(_styles.
   onDismiss: noop,
   onShare: function onShare() {},
   sharedBanks: (0, _immutable.List)(),
-  subAccountBankSharingEnabled: false,
   updateSharedCount: Function.prototype
 }, _SharingModalPresenter)) || _class);
 exports.SharingModalPresenter = SharingModalPresenter;

package/lib/common/components/resources/item/ItemEdit/presenter.js

@@ -23,7 +23,6 @@ var _formatMessage = _interopRequireDefault(require("@instructure/quiz-i18n/es/f
 var _interactionTypePropsHelper = require("../../../../util/interactionTypePropsHelper.js");
 var _ElementsForSelectors = _interopRequireDefault(require("../../../../../common/util/ElementsForSelectors.js"));
 var _instUIMessages = require("../../../../util/instUIMessages.js");
-var _featureCheck = require("../../../../../common/util/featureCheck.js");
 var ItemEdit = /*#__PURE__*/function (_Component) {
   (0, _inherits2.default)(ItemEdit, _Component);
   var _super = (0, _createSuper2.default)(ItemEdit);
@@ -71,7 +70,7 @@ var ItemEdit = /*#__PURE__*/function (_Component) {
       var launchContextUuid = this.props.scope.launch_context_uuid;
       var launchContextTitle = this.props.scope.title;
       // If editing within a quiz, there is no account launch context
-      if ((0, _featureCheck.featureOn)('sub_account_bank_sharing') && rootAccountName) {
+      if (rootAccountName) {
         launchContexts.push({
           uuid: this.props.rootAccountUuid,
           name: rootAccountName

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@instructure/quiz-core",
-  "version": "20.36.1-rc.4+6b7d9ae90",
+  "version": "20.36.1-rc.6+1a10615e3",
   "license": "MIT",
   "description": "The Quiz React SDK by Instructure Inc.",
   "author": "Instructure, Inc. Engineering and Product Design",
@@ -46,11 +46,11 @@
     "@instructure/emotion": "^8.56.3",
     "@instructure/grading-utils": "^1.0.0",
     "@instructure/outcomes-ui": "^3.2.2",
-    "@instructure/quiz-common": "^20.36.1-rc.4+6b7d9ae90",
-    "@instructure/quiz-i18n": "^20.36.1-rc.4+6b7d9ae90",
-    "@instructure/quiz-interactions": "^20.36.1-rc.4+6b7d9ae90",
-    "@instructure/quiz-number-input": "^20.36.1-rc.4+6b7d9ae90",
-    "@instructure/quiz-rce": "^20.36.1-rc.4+6b7d9ae90",
+    "@instructure/quiz-common": "^20.36.1-rc.6+1a10615e3",
+    "@instructure/quiz-i18n": "^20.36.1-rc.6+1a10615e3",
+    "@instructure/quiz-interactions": "^20.36.1-rc.6+1a10615e3",
+    "@instructure/quiz-number-input": "^20.36.1-rc.6+1a10615e3",
+    "@instructure/quiz-rce": "^20.36.1-rc.6+1a10615e3",
     "@instructure/ui-a11y-content": "^8.56.3",
     "@instructure/ui-alerts": "^8.56.3",
     "@instructure/ui-avatar": "^8.56.3",
@@ -114,7 +114,7 @@
     "file-saver": "~2.0.5",
     "humps": "^2.0.0",
     "immutable": "^3.8.1",
-    "instructure-validations": "^20.36.1-rc.4+6b7d9ae90",
+    "instructure-validations": "^20.36.1-rc.6+1a10615e3",
     "ipaddr.js": "^1.5.4",
     "isomorphic-fetch": "^2.2.0",
     "isuuid": "^0.1.0",
@@ -164,7 +164,7 @@
     "jquery": "^2.2.3",
     "karma-junit-reporter": "^2.0.1",
     "most-subject": "^5.3.0",
-    "quiz-presets": "^20.36.1-rc.4+6b7d9ae90",
+    "quiz-presets": "^20.36.1-rc.6+1a10615e3",
     "react": "^16.8.6",
     "react-addons-test-utils": "^15.6.2",
     "react-dom": "^16.8.6",
@@ -180,5 +180,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "6b7d9ae9049db850bc77f2182c392dfeda3844df"
+  "gitHead": "1a10615e378cbb095dce3fa4ddc808f3c9d629d0"
 }