@instructure/quiz-core 20.34.0 → 20.34.1-snapshot.10

package/es/reporting/components/resources/StudentAnalysis/ReportCard/presenter.js

@@ -8,16 +8,15 @@ import { Text } from '@instructure/ui-text';
 import { Link } from '@instructure/ui-link';
 import { Flex } from '@instructure/ui-flex';
 import { ScreenReaderContent } from '@instructure/ui-a11y-content';
-import { withStyle } from '@instructure/emotion';
 import t from '@instructure/quiz-i18n/es/format-message';
 import Card from "../../../../../common/components/shared/Card/index.js";
 import { generateStyle } from "./styles.js";
 import { generateComponentTheme } from "./theme.js";
 import { ExportButton } from "../../common/ExportButton/index.js";
 import Spinner from "../../../../../common/components/shared/spinner/Spinner.js";
-import { STUDENT_ANALYSES } from '@instructure/quiz-common';
+import { STUDENT_ANALYSES, withStyleOverrides } from '@instructure/quiz-common';
 import { AnalysisUpdateDate } from "../../common/AnalysisUpdateDate/index.js";
-export var StudentAnalysisReportCardPresenter = (_dec = withStyle(generateStyle, generateComponentTheme), _dec(_class = (_StudentAnalysisReportCardPresenter = /*#__PURE__*/function (_Component) {
+export var StudentAnalysisReportCardPresenter = (_dec = withStyleOverrides(generateStyle, generateComponentTheme), _dec(_class = (_StudentAnalysisReportCardPresenter = /*#__PURE__*/function (_Component) {
   _inherits(StudentAnalysisReportCardPresenter, _Component);
   var _super = _createSuper(StudentAnalysisReportCardPresenter);
   function StudentAnalysisReportCardPresenter() {
@@ -42,11 +41,14 @@ export var StudentAnalysisReportCardPresenter = (_dec = withStyle(generateStyle,
         "data-automation": "sdk-new-sa-report-card-spinner"
       }))));
     };
+    _this.getAttemptType = function () {
+      return _this.props.allAttemptsEnabled ? 'all_attempts' : 'last_attempt';
+    };
     _this.handleExport = function () {
-      _this.props.exportReport('csv', 'last_attempt', STUDENT_ANALYSES);
+      _this.props.exportReport('csv', _this.getAttemptType(), STUDENT_ANALYSES);
     };
     _this.handleGeneration = function () {
-      _this.props.generateReport('last_attempt', STUDENT_ANALYSES);
+      _this.props.generateReport(_this.getAttemptType(), STUDENT_ANALYSES);
     };
     _this.renderExportButton = function () {
       var _this$props = _this.props,
@@ -148,7 +150,7 @@ export var StudentAnalysisReportCardPresenter = (_dec = withStyle(generateStyle,
   }]);
   StudentAnalysisReportCardPresenter.displayName = "StudentAnalysisReportCardPresenter";
   return StudentAnalysisReportCardPresenter;
-}(Component), _StudentAnalysisReportCardPresenter.defaultProps = {
+}(Component), _StudentAnalysisReportCardPresenter.displayName = 'StudentAnalysisReportCardPresenter', _StudentAnalysisReportCardPresenter.componentId = "Quizzes".concat(_StudentAnalysisReportCardPresenter.displayName), _StudentAnalysisReportCardPresenter.defaultProps = {
   studentAnalyses: void 0,
   analysisError: void 0,
   generationInProgress: void 0

package/es/reporting/components/resources/StudentAnalysis/withStudentAnalysisReportState.js

@@ -22,7 +22,8 @@ function mapStateToProps(state, ownProps) {
     fetchStudentAnalyses: fetchStudentAnalyses,
     fetchAnalysesStatus: fetchAnalysesStatus,
     pollingInterval: ownProps.pollingInterval || DEFAULT_POLLING_INTERVAL,
-    exportEnabled: featureOn('student_analysis_csv')
+    exportEnabled: featureOn('student_analysis_csv'),
+    allAttemptsEnabled: featureOn('student_analysis_multiple_attempts')
   };
 }
 export function mapDispatchToProps(dispatch, ownProps) {

package/es/reporting/components/resources/StudentAnalysis/withStudentAnalysisReportStatePresenter.js

@@ -12,6 +12,7 @@ import { fetchReport } from "./fetchReport.js";
 import { fetchReportStatus } from "./fetchReportStatus.js";
 import { STUDENT_ANALYSES } from '@instructure/quiz-common';
 import { studentAnalysesPropType } from "../common/propTypes.js";
+import { featureOn } from "../../../../common/util/featureCheck.js";
 export var withStudentAnalysisReportStatePresenter = function withStudentAnalysisReportStatePresenter(OriginalComponent) {
   var StudentAnalysisReportStatePresenter = function StudentAnalysisReportStatePresenter(_ref) {
     var studentAnalyses = _ref.studentAnalyses,
@@ -69,6 +70,9 @@ export var withStudentAnalysisReportStatePresenter = function withStudentAnalysi
       _generateReport = _asyncToGenerator(/*#__PURE__*/_regeneratorRuntime.mark(function _callee3() {
         var filter,
           analysisType,
+          isMultiAttemptEnabled,
+          isStudentAnalysis,
+          selectedFilter,
           generationConfig,
           _args3 = arguments;
         return _regeneratorRuntime.wrap(function _callee3$(_context3) {
@@ -77,17 +81,20 @@ export var withStudentAnalysisReportStatePresenter = function withStudentAnalysi
               case 0:
                 filter = _args3.length > 0 && _args3[0] !== void 0 ? _args3[0] : 'last_attempt';
                 analysisType = _args3.length > 1 ? _args3[1] : void 0;
+                isMultiAttemptEnabled = featureOn('student_analysis_multiple_attempts');
+                isStudentAnalysis = analysisType === STUDENT_ANALYSES;
+                selectedFilter = isMultiAttemptEnabled && isStudentAnalysis ? 'all_attempts' : filter;
                 generationConfig = {
-                  filter: filter,
+                  filter: selectedFilter,
                   analysisType: analysisType,
                   setGenerationInProgress: setGenerationInProgress,
                   setStatusPollingInProgress: setStatusPollingInProgress,
                   generateAnalysis: generateAnalysis,
                   addError: addError
                 };
-                _context3.next = 5;
+                _context3.next = 8;
                 return triggerReportGeneration(generationConfig);
-              case 5:
+              case 8:
               case "end":
                 return _context3.stop();
             }
@@ -103,6 +110,9 @@ export var withStudentAnalysisReportStatePresenter = function withStudentAnalysi
       _exportReport = _asyncToGenerator(/*#__PURE__*/_regeneratorRuntime.mark(function _callee4(format) {
         var filter,
           analysisType,
+          isMultiAttemptEnabled,
+          isStudentAnalysis,
+          selectedFilter,
           exportConfig,
           _args4 = arguments;
         return _regeneratorRuntime.wrap(function _callee4$(_context4) {
@@ -111,18 +121,21 @@ export var withStudentAnalysisReportStatePresenter = function withStudentAnalysi
               case 0:
                 filter = _args4.length > 1 && _args4[1] !== void 0 ? _args4[1] : 'last_attempt';
                 analysisType = _args4.length > 2 && _args4[2] !== void 0 ? _args4[2] : STUDENT_ANALYSES;
+                isMultiAttemptEnabled = featureOn('student_analysis_multiple_attempts');
+                isStudentAnalysis = analysisType === STUDENT_ANALYSES;
+                selectedFilter = isMultiAttemptEnabled && isStudentAnalysis ? 'all_attempts' : filter;
                 exportConfig = {
                   getAnalysesExport: getAnalysesExport,
                   format: format,
-                  filter: filter,
+                  filter: selectedFilter,
                   activeQuizId: activeQuizId,
                   addError: addError,
                   saveFile: saveFile,
                   analysisType: analysisType
                 };
-                _context4.next = 5;
+                _context4.next = 8;
                 return triggerReportExport(exportConfig);
-              case 5:
+              case 8:
               case "end":
                 return _context4.stop();
             }

package/lib/banks/api/banks.js

@@ -5,7 +5,7 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.getExistingBankAndEntries = getExistingBankAndEntries;
-exports.createBankCloneJob = exports.updateSharedBankPermission = exports.getSharedBanks = exports.getSharedBank = exports.shareBank = exports.deleteBank = exports.updateBank = exports.createBank = exports.getSpecificBanks = exports.getBank = exports.getAllBanks = exports.getBanks = void 0;
+exports.createBankCloneJob = exports.updateSharedBankPermission = exports.getSharedBanks = exports.getSharedBank = exports.shareBank = exports.deleteBank = exports.updateBank = exports.createBank = exports.getSpecificBanks = exports.getBank = exports.getAllBanks = exports.getBanks = exports.simpleFetchBanks = void 0;
 var _objectSpread2 = _interopRequireDefault(require("@babel/runtime/helpers/objectSpread2"));
 var _slicedToArray2 = _interopRequireDefault(require("@babel/runtime/helpers/slicedToArray"));
 var _Fetcher = _interopRequireDefault(require("../../common/util/Fetcher.js"));
@@ -24,10 +24,15 @@ var _lodash = require("lodash");
 var _uuid = require("uuid");
 var banksApiEndpoint = '/api/banks';
 var DEFAULT_PAGE_SIZE = 100;
+var simpleFetchBanks = function simpleFetchBanks() {
+  var params = arguments.length > 0 && arguments[0] !== void 0 ? arguments[0] : {};
+  var fetcher = new _PaginatedFetcher.default();
+  return fetcher.get(banksApiEndpoint, params);
+};
+exports.simpleFetchBanks = simpleFetchBanks;
 var _fetchBanks = function _fetchBanks(dispatch) {
   var params = arguments.length > 1 && arguments[1] !== void 0 ? arguments[1] : {};
-  var fetcher = new _PaginatedFetcher.default();
-  return fetcher.get(banksApiEndpoint, params).then(function (_ref) {
+  return simpleFetchBanks(params).then(function (_ref) {
     var _ref2 = (0, _slicedToArray2.default)(_ref, 2),
       response = _ref2[0],
       pageData = _ref2[1];

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

@@ -16,16 +16,12 @@ var _presenter = _interopRequireDefault(require("./presenter.js"));
 var modalActions = _interopRequireWildcard(require("../../../common/actions/modal.js"));
 var _banks2 = require("../../../common/actions/banks.js");
 var uiActions = _interopRequireWildcard(require("../../../common/actions/ui.js"));
-var _Bank = _interopRequireDefault(require("../../../common/records/Bank.js"));
 var _quizCommon = require("@instructure/quiz-common");
 var _modalHelpers = require("../../../common/util/modalHelpers.js");
 function mapStateToProps(state, props) {
-  var selectedBankOption = state.getIn(['ui', _quizCommon.ADD_TO_BANK_MODAL_BANK_SELECTION]);
-  var selectedBank = selectedBankOption && new _Bank.default(state.getIn(['banks', selectedBankOption]));
   return {
     canvasOrigin: state.getIn(['config', 'canvasOrigin']),
     checkContentShareability: props.entryType === 'Item' ? _items.checkItemShareability : _stimuli.checkStimulusShareability,
-    selectedBank: selectedBank,
     modalTrigger: (0, _modalHelpers.getModalTrigger)(state),
     modalOpen: state.getIn(['modal', 'isOpen']) === [_quizCommon.ADD_TO_BANK_MODAL, props.entryType, props.entryId].join(':')
   };

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

@@ -21,7 +21,7 @@ var _uiTextInput = require("@instructure/ui-text-input");
 var _uiRadioInput = require("@instructure/ui-radio-input");
 var _uiA11yContent = require("@instructure/ui-a11y-content");
 var _Bank = _interopRequireDefault(require("../../../common/records/Bank.js"));
-var _index = _interopRequireDefault(require("../../../common/components/resources/BankSelection/index.js"));
+var _presenter = _interopRequireDefault(require("../../../common/components/resources/BankSelection/presenter.js"));
 var _uiSpinner = require("@instructure/ui-spinner");
 var _uiText = require("@instructure/ui-text");
 var _uiIcons = require("@instructure/ui-icons");
@@ -53,7 +53,8 @@ var AddToBankModal = /*#__PURE__*/function (_Component) {
     _this.state = {
       bankType: 'existing',
       contentShareable: 'loading',
-      newBankTitle: ''
+      newBankTitle: '',
+      selectedBank: null
     };
     _this.copyBankEntry = function (entryId, bankId) {
       if ((0, _featureCheck.featureOn)('instfs_bank_entry_movement_endpoints')) {
@@ -88,14 +89,16 @@ var AddToBankModal = /*#__PURE__*/function (_Component) {
           _this.addToBank(bankData.id);
         });
       } else {
-        _this.addToBank(_this.props.selectedBank.id);
+        _this.addToBank(_this.state.selectedBank.id);
       }
     };
     _this.closeAction = function () {
       _this.props.closeModal();
     };
     _this.onBankSelection = function (e, option) {
-      _this.props.uiSet(_quizCommon.ADD_TO_BANK_MODAL_BANK_SELECTION, option.value);
+      _this.setState({
+        selectedBank: new _Bank.default(option)
+      });
     };
     _this.handleRadioChange = function (event, value) {
       _this.setState({
@@ -176,7 +179,7 @@ var AddToBankModal = /*#__PURE__*/function (_Component) {
         newBankTitle = _this$state.newBankTitle,
         contentShareable = _this$state.contentShareable;
       var contentShareableLoading = contentShareable === 'loading';
-      var disabled = bankType === 'existing' && !this.props.selectedBank || bankType === 'new' && (!newBankTitle || !newBankTitle.trim()) || contentShareableLoading;
+      var disabled = bankType === 'existing' && !this.state.selectedBank || bankType === 'new' && (!newBankTitle || !newBankTitle.trim()) || contentShareableLoading;
       return /*#__PURE__*/_react.default.createElement(_uiButtons.Button, {
         type: "button",
         color: "primary",
@@ -190,9 +193,9 @@ var AddToBankModal = /*#__PURE__*/function (_Component) {
     key: "renderBankTypeContent",
     value: function renderBankTypeContent() {
       if (this.state.bankType === 'existing') {
-        return /*#__PURE__*/_react.default.createElement(_index.default, {
+        return /*#__PURE__*/_react.default.createElement(_presenter.default, {
           onBankSelection: this.onBankSelection,
-          selectedBank: this.props.selectedBank,
+          selectedBank: this.state.selectedBank,
           "data-automation": "sdk-bank-selection"
         });
       }
@@ -292,7 +295,6 @@ AddToBankModal.propTypes = {
   modalOpen: _propTypes.default.bool.isRequired,
   onAdd: _propTypes.default.func,
   onContinue: _propTypes.default.func,
-  selectedBank: _reactImmutableProptypes.default.record,
   uiSet: _propTypes.default.func.isRequired,
   workingEntry: _reactImmutableProptypes.default.record.isRequired,
   quizId: _propTypes.default.string.isRequired
@@ -301,8 +303,7 @@ AddToBankModal.defaultProps = {
   canvasOrigin: '',
   handleValidationErrorsFromApi: function handleValidationErrorsFromApi() {},
   onAdd: function onAdd() {},
-  onContinue: function onContinue() {},
-  selectedBank: null
+  onContinue: function onContinue() {}
 };
 AddToBankModal.contextTypes = {
   locale: _propTypes.default.string

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

@@ -161,7 +161,7 @@ var AsyncSearch = /*#__PURE__*/function (_Component) {
           });
         });
       }
-      if (_this.props.improvedBankSharingEnabled && shouldSearchCourses) {
+      if (shouldSearchCourses) {
         _this.props.getShareableCourses(value).then(function (shareableCourses) {
           var filteredCourses = _this.filterOptions(shareableCourses);
           _this.setState({
@@ -366,7 +366,6 @@ var AsyncSearch = /*#__PURE__*/function (_Component) {
 exports.AsyncSearch = AsyncSearch;
 AsyncSearch.propTypes = {
   debounceDelay: _propTypes.default.number,
-  improvedBankSharingEnabled: _propTypes.default.bool.isRequired,
   minChars: _propTypes.default.number,
   onSelectionChange: _propTypes.default.func.isRequired,
   getShareableCourses: _propTypes.default.func,

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

@@ -45,7 +45,7 @@ var _theme = _interopRequireDefault(require("./theme.js"));
 var _dec, _class, _Bank;
 /** @jsx jsx */
 var _ref = (0, _emotion.jsx)(_uiIcons.IconMoreSolid, null);
-var Bank = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default), _dec(_class = (_Bank = /*#__PURE__*/function (_Component) {
+var Bank = (_dec = (0, _quizCommon.withStyleOverrides)(_styles.default, _theme.default), _dec(_class = (_Bank = /*#__PURE__*/function (_Component) {
   (0, _inherits2.default)(Bank, _Component);
   var _super = (0, _createSuper2.default)(Bank);
   function Bank() {
@@ -448,7 +448,7 @@ var Bank = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default), _de
   }]);
   Bank.displayName = "Bank";
   return Bank;
-}(_react.Component), _Bank.propTypes = {
+}(_react.Component), _Bank.displayName = 'Bank', _Bank.componentId = "Quizzes".concat(_Bank.displayName), _Bank.propTypes = {
   activeQuizId: _propTypes.default.string,
   addEntryToQuiz: _propTypes.default.func,
   bankId: _propTypes.default.string.isRequired,

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

@@ -17,9 +17,10 @@ var _index = _interopRequireDefault(require("../../../common/components/shared/C
 var _index2 = _interopRequireDefault(require("../BankEntryRow/index.js"));
 var _styles = _interopRequireDefault(require("./styles.js"));
 var _theme = _interopRequireDefault(require("./theme.js"));
+var _quizCommon = require("@instructure/quiz-common");
 var _dec, _class, _BankEntriesList;
 /** @jsx jsx */
-var BankEntriesList = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default), _dec(_class = (_BankEntriesList = /*#__PURE__*/function (_Component) {
+var BankEntriesList = (_dec = (0, _quizCommon.withStyleOverrides)(_styles.default, _theme.default), _dec(_class = (_BankEntriesList = /*#__PURE__*/function (_Component) {
   (0, _inherits2.default)(BankEntriesList, _Component);
   var _super = (0, _createSuper2.default)(BankEntriesList);
   function BankEntriesList() {
@@ -52,7 +53,7 @@ var BankEntriesList = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.de
   }]);
   BankEntriesList.displayName = "BankEntriesList";
   return BankEntriesList;
-}(_react.Component), _BankEntriesList.propTypes = {
+}(_react.Component), _BankEntriesList.displayName = 'BankEntriesList', _BankEntriesList.componentId = "Quizzes".concat(_BankEntriesList.displayName), _BankEntriesList.propTypes = {
   addEntryToQuiz: _propTypes.default.func,
   bank: _reactImmutableProptypes.default.record.isRequired,
   bankEntries: _reactImmutableProptypes.default.list.isRequired,

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

@@ -52,7 +52,7 @@ var _theme = _interopRequireDefault(require("./theme.js"));
 var _dec, _class, _BankEntry;
 /** @jsx jsx */
 var _ref3 = (0, _emotion.jsx)(_Spinner.default, null);
-var BankEntry = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default), _dec(_class = (_BankEntry = /*#__PURE__*/function (_Component) {
+var BankEntry = (_dec = (0, _quizCommon.withStyleOverrides)(_styles.default, _theme.default), _dec(_class = (_BankEntry = /*#__PURE__*/function (_Component) {
   (0, _inherits2.default)(BankEntry, _Component);
   var _super = (0, _createSuper2.default)(BankEntry);
   function BankEntry() {
@@ -588,7 +588,7 @@ var BankEntry = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default)
   }]);
   BankEntry.displayName = "BankEntry";
   return BankEntry;
-}(_react.Component), _BankEntry.propTypes = {
+}(_react.Component), _BankEntry.displayName = 'BankEntry', _BankEntry.componentId = "Quizzes".concat(_BankEntry.displayName), _BankEntry.propTypes = {
   bankEntryId: _propTypes.default.string.isRequired,
   bankId: _propTypes.default.string.isRequired,
   bank: _reactImmutableProptypes.default.record.isRequired,

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

@@ -39,7 +39,7 @@ var _index6 = _interopRequireDefault(require("../../../common/components/shared/
 var _dec, _class, _BankEntryRow;
 /** @jsx jsx */
 var _ref2 = (0, _emotion.jsx)("div", null);
-var BankEntryRow = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default), _dec(_class = (_BankEntryRow = /*#__PURE__*/function (_withI18nSupport) {
+var BankEntryRow = (_dec = (0, _quizCommon.withStyleOverrides)(_styles.default, _theme.default), _dec(_class = (_BankEntryRow = /*#__PURE__*/function (_withI18nSupport) {
   (0, _inherits2.default)(BankEntryRow, _withI18nSupport);
   var _super = (0, _createSuper2.default)(BankEntryRow);
   function BankEntryRow() {
@@ -349,7 +349,7 @@ var BankEntryRow = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.defau
   }]);
   BankEntryRow.displayName = "BankEntryRow";
   return BankEntryRow;
-}((0, _quizCommon.withI18nSupport)(_react.Component)), _BankEntryRow.propTypes = {
+}((0, _quizCommon.withI18nSupport)(_react.Component)), _BankEntryRow.displayName = 'BankEntryRow', _BankEntryRow.componentId = "Quizzes".concat(_BankEntryRow.displayName), _BankEntryRow.propTypes = {
   activeQuizId: _propTypes.default.string,
   addEntryToQuiz: _propTypes.default.func,
   bankEntry: _reactImmutableProptypes.default.record.isRequired,

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

@@ -27,6 +27,7 @@ var _index = _interopRequireDefault(require("../../../common/components/shared/T
 var _interactionTypeMetadata = require("../../../common/util/interactionTypeMetadata.js");
 var _styles = _interopRequireDefault(require("./styles.js"));
 var _theme = _interopRequireDefault(require("./theme.js"));
+var _quizCommon = require("@instructure/quiz-common");
 var _dec, _class, _BankSearch;
 /** @jsx jsx */
 var tagShape = _propTypes.default.shape({
@@ -36,7 +37,7 @@ var tagShape = _propTypes.default.shape({
   }).isRequired,
   count: _propTypes.default.number.isRequired
 });
-var BankSearch = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default), _dec(_class = (_BankSearch = /*#__PURE__*/function (_Component) {
+var BankSearch = (_dec = (0, _quizCommon.withStyleOverrides)(_styles.default, _theme.default), _dec(_class = (_BankSearch = /*#__PURE__*/function (_Component) {
   (0, _inherits2.default)(BankSearch, _Component);
   var _super = (0, _createSuper2.default)(BankSearch);
   function BankSearch() {
@@ -163,7 +164,7 @@ var BankSearch = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default
   }]);
   BankSearch.displayName = "BankSearch";
   return BankSearch;
-}(_react.Component), _BankSearch.propTypes = {
+}(_react.Component), _BankSearch.displayName = 'BankSearch', _BankSearch.componentId = "Quizzes".concat(_BankSearch.displayName), _BankSearch.propTypes = {
   allInteractions: _propTypes.default.bool,
   interactionTypes: _propTypes.default.arrayOf(_propTypes.default.shape({
     count: _propTypes.default.number,

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

@@ -25,7 +25,6 @@ function mapStateToProps(state, props) {
   var rootAccountUuid = state.getIn(['outcomes', 'externalAccountUuid']);
   return {
     itemBankSearchTableEnabled: (0, _featureCheck.featureOn)('item_bank_search_table'),
-    improvedBankSharingEnabled: (0, _featureCheck.featureOn)('improved_bank_sharing'),
     usedByCourseEnabled: (0, _featureCheck.featureOn)('filter_item_banks_used_by_course'),
     subAccountBankSharingEnabled: (0, _featureCheck.featureOn)('sub_account_bank_sharing'),
     scopeType: scopeType,

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

@@ -47,7 +47,7 @@ var sortOrderIcons = {
 var _ref4 = (0, _emotion.jsx)(_uiImg.Img, {
   src: _noResults.default
 });
-var Banks = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default), _dec(_class = (_Banks = /*#__PURE__*/function (_Component) {
+var Banks = (_dec = (0, _quizCommon.withStyleOverrides)(_styles.default, _theme.default), _dec(_class = (_Banks = /*#__PURE__*/function (_Component) {
   (0, _inherits2.default)(Banks, _Component);
   var _super = (0, _createSuper2.default)(Banks);
   function Banks(props) {
@@ -316,7 +316,6 @@ var Banks = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default), _d
         width: 4
       }, (0, _emotion.jsx)(_index7.default, {
         getShareableSubaccounts: this.props.getShareableSubaccounts,
-        improvedBankSharingEnabled: this.props.improvedBankSharingEnabled,
         minChars: this.props.minChars,
         onSelectionChange: this.handleSearchSelectionChange,
         searchType: _quizCommon.SHARED_WITH_ACCOUNT,
@@ -376,7 +375,7 @@ var Banks = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default), _d
     key: "renderHeader",
     value: function renderHeader() {
       return (0, _emotion.jsx)("div", {
-        css: [this.props.styles.header, !this.props.improvedBankSharingEnabled ? this.props.styles.borderedBottom : {}]
+        css: [this.props.styles.header]
       }, (0, _emotion.jsx)(_uiHeading.Heading, {
         as: this.props.isCompact ? 'h2' : 'h1',
         level: "h1"
@@ -431,14 +430,14 @@ var Banks = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default), _d
         returnToUrl: this.props.returnToUrl,
         scope: this.props.scope,
         navigateToBuild: this.props.navigateToBuild
-      })), this.renderHeader(), this.props.improvedBankSharingEnabled && this.renderFilters(), this.renderBanksSearchResults(), (0, _emotion.jsx)(_index5.default, {
+      })), this.renderHeader(), this.renderFilters(), this.renderBanksSearchResults(), (0, _emotion.jsx)(_index5.default, {
         itemBanksScope: this.props.scope
       })));
     }
   }]);
   Banks.displayName = "Banks";
   return Banks;
-}(_react.Component), _Banks.propTypes = {
+}(_react.Component), _Banks.displayName = 'Banks', _Banks.componentId = "Quizzes".concat(_Banks.displayName), _Banks.propTypes = {
   closeTray: _propTypes.default.func,
   createBank: _propTypes.default.func.isRequired,
   currentUser: _propTypes.default.shape({
@@ -452,7 +451,6 @@ var Banks = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default), _d
   }),
   // from consumer
   clear: _propTypes.default.func.isRequired,
-  improvedBankSharingEnabled: _propTypes.default.bool,
   getBanks: _propTypes.default.func.isRequired,
   getCourseInfo: _propTypes.default.func,
   // from consumer
@@ -520,7 +518,6 @@ var Banks = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default), _d
   onShare: function onShare() {},
   scope: {},
   returnToUrl: null,
-  improvedBankSharingEnabled: false,
   scopeType: 'user',
   sortBy: 'name',
   sortOrder: 'ascending',

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

@@ -12,7 +12,6 @@ var _presenter = _interopRequireDefault(require("./presenter.js"));
 var _Bank = _interopRequireDefault(require("../../../common/records/Bank.js"));
 var _modal = require("../../../common/actions/modal.js");
 var _banks = require("../../api/banks.js");
-var _featureCheck = require("../../../common/util/featureCheck.js");
 var _alerts = require("../../../common/actions/alerts.js");
 var _copyingBanksPoller = require("../../api/copyingBanksPoller.js");
 function mapStateToProps(state, props) {
@@ -20,8 +19,7 @@ function mapStateToProps(state, props) {
     appContainer: state.getIn(['ui', 'appContainer']),
     banks: (0, _immutable.List)(props.recordIds).map(function (id) {
       return _Bank.default.create(state.getIn(['banks', id]));
-    }),
-    duplicateItemBanks: (0, _featureCheck.featureOn)('duplicate_item_banks')
+    })
   };
 }
 var _default = (0, _reactRedux.connect)(mapStateToProps, {

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

@@ -41,7 +41,7 @@ var _ref4 = (0, _emotion.jsx)(_uiTable.Table.Cell, null);
 var _ref5 = (0, _emotion.jsx)(_uiTable.Table.Cell, null);
 var _ref6 = (0, _emotion.jsx)(_uiTable.Table.Cell, null);
 var _ref7 = (0, _emotion.jsx)(_uiTable.Table.Cell, null);
-var BanksList = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default), _dec(_class = (_BanksList = /*#__PURE__*/function (_withI18nSupport) {
+var BanksList = (_dec = (0, _quizCommon.withStyleOverrides)(_styles.default, _theme.default), _dec(_class = (_BanksList = /*#__PURE__*/function (_withI18nSupport) {
   (0, _inherits2.default)(BanksList, _withI18nSupport);
   var _super = (0, _createSuper2.default)(BanksList);
   function BanksList() {
@@ -271,21 +271,6 @@ var BanksList = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default)
           name: bank.title
         })
       }));
-      var duplicateBankOption = (0, _emotion.jsx)(_uiButtons.IconButton, {
-        onClick: _this.openDuplicateBankModal(bank),
-        name: "Duplicate",
-        size: "small",
-        "data-automation": "sdk-duplicate-bank-button",
-        renderIcon: _uiIcons.IconCopyLine,
-        withBackground: false,
-        withBorder: false,
-        title: (0, _formatMessage.default)('Duplicate bank {name}', {
-          name: bank.title
-        }),
-        screenReaderLabel: (0, _formatMessage.default)('Duplicate bank {name}', {
-          name: bank.title
-        })
-      });
       if (bank.status === 'copying') {
         return (0, _emotion.jsx)("div", {
           css: _this.props.styles.bankOptions
@@ -297,7 +282,21 @@ var BanksList = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default)
       } else {
         return (0, _emotion.jsx)("div", {
           css: _this.props.styles.bankOptions
-        }, _this.props.duplicateItemBanks ? duplicateBankOption : null, (0, _emotion.jsx)(_index.default, {
+        }, (0, _emotion.jsx)(_uiButtons.IconButton, {
+          onClick: _this.openDuplicateBankModal(bank),
+          name: "Duplicate",
+          size: "small",
+          "data-automation": "sdk-duplicate-bank-button",
+          renderIcon: _uiIcons.IconCopyLine,
+          withBackground: false,
+          withBorder: false,
+          title: (0, _formatMessage.default)('Duplicate bank {name}', {
+            name: bank.title
+          }),
+          screenReaderLabel: (0, _formatMessage.default)('Duplicate bank {name}', {
+            name: bank.title
+          })
+        }), (0, _emotion.jsx)(_index.default, {
           name: "item_bank_sharing"
         }, (0, _emotion.jsx)(_uiButtons.IconButton, {
           onClick: _this.openSharingModal(bank),
@@ -470,7 +469,7 @@ var BanksList = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default)
   }]);
   BanksList.displayName = "BanksList";
   return BanksList;
-}((0, _quizCommon.withI18nSupport)(_react.Component)), _BanksList.propTypes = {
+}((0, _quizCommon.withI18nSupport)(_react.Component)), _BanksList.displayName = 'BanksList', _BanksList.componentId = "Quizzes".concat(_BanksList.displayName), _BanksList.propTypes = {
   addInfo: _propTypes.default.func.isRequired,
   addToPolling: _propTypes.default.func.isRequired,
   appContainer: _CustomPropTypes.default.selectors.isRequired,
@@ -487,7 +486,6 @@ var BanksList = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default)
   }),
   focusCreateBankButton: _propTypes.default.func.isRequired,
   deleteBank: _propTypes.default.func.isRequired,
-  duplicateItemBanks: _propTypes.default.bool,
   getCourseInfo: _propTypes.default.func,
   getUserInfo: _propTypes.default.func,
   getShareableCourses: _propTypes.default.func,
@@ -511,8 +509,7 @@ var BanksList = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default)
   minChars: void 0,
   locale: null,
   timezone: null,
-  onShare: function onShare() {},
-  duplicateItemBanks: false
+  onShare: function onShare() {}
 }, _BanksList.contextTypes = {
   locale: _propTypes.default.string,
   timezone: _propTypes.default.string

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

@@ -14,15 +14,11 @@ var _presenter = _interopRequireDefault(require("./presenter.js"));
 var modalActions = _interopRequireWildcard(require("../../../common/actions/modal.js"));
 var _banks2 = require("../../../common/actions/banks.js");
 var uiActions = _interopRequireWildcard(require("../../../common/actions/ui.js"));
-var _Bank = _interopRequireDefault(require("../../../common/records/Bank.js"));
 var _quizCommon = require("@instructure/quiz-common");
 var _modalHelpers = require("../../../common/util/modalHelpers.js");
 function mapStateToProps(state, props) {
-  var selectedBankOption = state.getIn(['ui', _quizCommon.COPY_MOVE_BANK_ENTRY_MODAL_BANK_SELECTION]);
-  var selectedBank = selectedBankOption && new _Bank.default(state.getIn(['banks', selectedBankOption]));
   var modalKey = [_quizCommon.COPY_MOVE_BANK_ENTRY_MODAL, props.entryType, props.entryId].join(':');
   return {
-    selectedBank: selectedBank,
     modalTrigger: (0, _modalHelpers.getModalTrigger)(state),
     keepCopy: state.getIn(['ui', _quizCommon.COPY_MOVE_BANK_ENTRY_MODAL_KEEP_COPY], true),
     modalOpen: state.getIn(['modal', 'isOpen']) === modalKey