@instructure/quiz-core 17.45.1-rc.1 → 17.45.1

package/CHANGELOG.md

@@ -3,6 +3,17 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [17.45.1](https://gerrit.instructure.com/quizzes-ui/compare/v17.45.0...v17.45.1) (2022-12-20)
+
+
+### Bug Fixes
+
+* **RCE:** prevent double requests with use_new_rce enabled ([a59dd40](https://gerrit.instructure.com/quizzes-ui/commits/a59dd404e5e28b14b94b79ec1526be2d44e816cb))
+
+
+
+
+
 # [17.45.0](https://gerrit.instructure.com/quizzes-ui/compare/v17.44.0...v17.45.0) (2022-12-13)
 
 

package/es/common/actions/ui.js

@@ -1,4 +1,4 @@
-import { CLEAR_UI, MERGE_UI, SET_UI, TOGGLE_UI, OCCLUDE_QUIZ_ENTRIES, ACTIVE_BANK_CHANGE, ACTIVE_BANK_ENTRY_CHANGE } from '@instructure/quiz-common';
+import { CLEAR_UI, MERGE_UI, SET_UI, TOGGLE_UI, OCCLUDE_QUIZ_ENTRIES, ACTIVE_BANK_CHANGE, ACTIVE_BANK_ENTRY_CHANGE, LOADED_API_FEATURES } from '@instructure/quiz-common';
 export var clear = function clear(key) {
   return {
     payload: key,
@@ -47,6 +47,12 @@ export var newActiveBankEntry = function newActiveBankEntry(activeBankEntryId) {
     payload: activeBankEntryId
   };
 };
+export var loadedApiFeatures = function loadedApiFeatures(loaded) {
+  return {
+    type: LOADED_API_FEATURES,
+    payload: loaded
+  };
+};
 export var setActiveCalculatorId = function setActiveCalculatorId(id) {
   return set('activeCalculatorId', id);
 };

package/es/common/api/features.js

@@ -1,5 +1,6 @@
 import Fetcher from "../util/Fetcher.js";
 import { addFeatures } from "../actions/features.js";
+import { loadedApiFeatures } from "../actions/ui.js";
 export var getFeatures = function getFeatures() {
   var url = '/api/features';
   return function (dispatch, getState) {
@@ -9,6 +10,8 @@ export var getFeatures = function getFeatures() {
       dispatch(addFeatures(featuresList));
     };
 
-    return fetcher.get(url).then(addFeaturesToStore);
+    return fetcher.get(url).then(addFeaturesToStore).then(function () {
+      return dispatch(loadedApiFeatures(true));
+    });
   };
 };

package/es/common/reducers/ui.js

@@ -1,5 +1,5 @@
 import { fromJS } from 'immutable';
-import { ACTIVE_BANK_CHANGE, ACTIVE_BANK_ENTRY_CHANGE, CLEAR_UI, MERGE_UI, SET_UI, TOGGLE_UI, OCCLUDE_QUIZ_ENTRIES } from '@instructure/quiz-common';
+import { ACTIVE_BANK_CHANGE, ACTIVE_BANK_ENTRY_CHANGE, CLEAR_UI, MERGE_UI, SET_UI, TOGGLE_UI, OCCLUDE_QUIZ_ENTRIES, LOADED_API_FEATURES } from '@instructure/quiz-common';
 
 var toggle = function toggle(state, key) {
   return state.update(key, function (value) {
@@ -34,6 +34,9 @@ export default (function () {
     case ACTIVE_BANK_ENTRY_CHANGE:
       return state.set('activeBankEntryId', action.payload);
 
+    case LOADED_API_FEATURES:
+      return state.set('loadedApiFeatures', action.payload);
+
     default:
       return state;
   }

package/es/common/util/RceConfigProvider.js

@@ -185,10 +185,20 @@ export var RceConfigProviderComponent = function RceConfigProviderComponent(prop
       }, _callee2, null, [[2, 21, 25, 28]]);
     }))();
   }, [props.useNewRce]);
-  if (!props.useNewRce) return /*#__PURE__*/React.createElement(React.Fragment, null, props.children);
-  return canvasConfigLoading ? _ref4 : /*#__PURE__*/React.createElement(RceConfigContext.Provider, {
-    value: getConfig()
-  }, props.children);
+
+  if (props.loadedApiFeatures) {
+    if (props.useNewRce) {
+      if (!canvasConfigLoading) {
+        return /*#__PURE__*/React.createElement(RceConfigContext.Provider, {
+          value: getConfig()
+        }, props.children);
+      }
+    } else {
+      return /*#__PURE__*/React.createElement(React.Fragment, null, props.children);
+    }
+  }
+
+  return _ref4;
 };
 RceConfigProviderComponent.propTypes = {
   children: PropTypes.node.isRequired,
@@ -200,7 +210,8 @@ RceConfigProviderComponent.propTypes = {
   rcsHost: PropTypes.string,
   rcsJwt: PropTypes.string,
   rcsJwtRefreshFn: PropTypes.func,
-  canvasOrigin: PropTypes.string
+  canvasOrigin: PropTypes.string,
+  loadedApiFeatures: PropTypes.bool
 };
 RceConfigProviderComponent.defaultProps = {
   contextId: null,
@@ -209,12 +220,14 @@ RceConfigProviderComponent.defaultProps = {
   rcsHost: null,
   rcsJwt: null,
   rcsJwtRefreshFn: null,
-  canvasOrigin: null
+  canvasOrigin: null,
+  loadedApiFeatures: false
 };
 
-var mapStateToProps = function mapStateToProps() {
+var mapStateToProps = function mapStateToProps(state, ownProps) {
   return {
-    useNewRce: isNewRCEEnabled()
+    useNewRce: isNewRCEEnabled(),
+    loadedApiFeatures: state.getIn(['ui', 'loadedApiFeatures']) || ownProps.loadedApiFeatures
   };
 };
 

package/lib/common/actions/ui.js

@@ -3,7 +3,7 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.setActiveCalculatorId = exports.newActiveBankEntry = exports.newActiveBank = exports.occludeQuizEntries = exports.merge = exports.set = exports.toggle = exports.clear = void 0;
+exports.setActiveCalculatorId = exports.loadedApiFeatures = exports.newActiveBankEntry = exports.newActiveBank = exports.occludeQuizEntries = exports.merge = exports.set = exports.toggle = exports.clear = void 0;
 
 var _quizCommon = require("@instructure/quiz-common");
 
@@ -76,6 +76,15 @@ var newActiveBankEntry = function newActiveBankEntry(activeBankEntryId) {
 
 exports.newActiveBankEntry = newActiveBankEntry;
 
+var loadedApiFeatures = function loadedApiFeatures(loaded) {
+  return {
+    type: _quizCommon.LOADED_API_FEATURES,
+    payload: loaded
+  };
+};
+
+exports.loadedApiFeatures = loadedApiFeatures;
+
 var setActiveCalculatorId = function setActiveCalculatorId(id) {
   return set('activeCalculatorId', id);
 };

package/lib/common/api/features.js

@@ -11,6 +11,8 @@ var _Fetcher = _interopRequireDefault(require("../util/Fetcher.js"));
 
 var _features = require("../actions/features.js");
 
+var _ui = require("../actions/ui.js");
+
 var getFeatures = function getFeatures() {
   var url = '/api/features';
   return function (dispatch, getState) {
@@ -20,7 +22,9 @@ var getFeatures = function getFeatures() {
       dispatch((0, _features.addFeatures)(featuresList));
     };
 
-    return fetcher.get(url).then(addFeaturesToStore);
+    return fetcher.get(url).then(addFeaturesToStore).then(function () {
+      return dispatch((0, _ui.loadedApiFeatures)(true));
+    });
   };
 };
 

package/lib/common/reducers/ui.js

@@ -43,6 +43,9 @@ var _default = function _default() {
     case _quizCommon.ACTIVE_BANK_ENTRY_CHANGE:
       return state.set('activeBankEntryId', action.payload);
 
+    case _quizCommon.LOADED_API_FEATURES:
+      return state.set('loadedApiFeatures', action.payload);
+
     default:
       return state;
   }

package/lib/common/util/RceConfigProvider.js

@@ -212,10 +212,20 @@ var RceConfigProviderComponent = function RceConfigProviderComponent(props) {
       }, _callee2, null, [[2, 21, 25, 28]]);
     }))();
   }, [props.useNewRce]);
-  if (!props.useNewRce) return /*#__PURE__*/_react.default.createElement(_react.default.Fragment, null, props.children);
-  return canvasConfigLoading ? _ref4 : /*#__PURE__*/_react.default.createElement(_quizCommon.RceConfigContext.Provider, {
-    value: getConfig()
-  }, props.children);
+
+  if (props.loadedApiFeatures) {
+    if (props.useNewRce) {
+      if (!canvasConfigLoading) {
+        return /*#__PURE__*/_react.default.createElement(_quizCommon.RceConfigContext.Provider, {
+          value: getConfig()
+        }, props.children);
+      }
+    } else {
+      return /*#__PURE__*/_react.default.createElement(_react.default.Fragment, null, props.children);
+    }
+  }
+
+  return _ref4;
 };
 
 exports.RceConfigProviderComponent = RceConfigProviderComponent;
@@ -229,7 +239,8 @@ RceConfigProviderComponent.propTypes = {
   rcsHost: _propTypes.default.string,
   rcsJwt: _propTypes.default.string,
   rcsJwtRefreshFn: _propTypes.default.func,
-  canvasOrigin: _propTypes.default.string
+  canvasOrigin: _propTypes.default.string,
+  loadedApiFeatures: _propTypes.default.bool
 };
 RceConfigProviderComponent.defaultProps = {
   contextId: null,
@@ -238,12 +249,14 @@ RceConfigProviderComponent.defaultProps = {
   rcsHost: null,
   rcsJwt: null,
   rcsJwtRefreshFn: null,
-  canvasOrigin: null
+  canvasOrigin: null,
+  loadedApiFeatures: false
 };
 
-var mapStateToProps = function mapStateToProps() {
+var mapStateToProps = function mapStateToProps(state, ownProps) {
   return {
-    useNewRce: (0, _rceChecker.default)()
+    useNewRce: (0, _rceChecker.default)(),
+    loadedApiFeatures: state.getIn(['ui', 'loadedApiFeatures']) || ownProps.loadedApiFeatures
   };
 };
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@instructure/quiz-core",
-  "version": "17.45.1-rc.1+652efed20",
+  "version": "17.45.1",
   "license": "MIT",
   "description": "The Quiz React SDK by Instructure Inc.",
   "author": "Instructure, Inc. Engineering and Product Design",
@@ -41,11 +41,11 @@
   ],
   "dependencies": {
     "@instructure/outcomes-ui": "^2.1.2",
-    "@instructure/quiz-common": "17.45.1-rc.1+652efed20",
-    "@instructure/quiz-i18n": "17.45.1-rc.1+652efed20",
-    "@instructure/quiz-interactions": "17.45.1-rc.1+652efed20",
-    "@instructure/quiz-number-input": "17.45.1-rc.1+652efed20",
-    "@instructure/quiz-rce": "17.45.1-rc.1+652efed20",
+    "@instructure/quiz-common": "^17.45.1",
+    "@instructure/quiz-i18n": "^17.45.1",
+    "@instructure/quiz-interactions": "^17.45.1",
+    "@instructure/quiz-number-input": "^17.45.1",
+    "@instructure/quiz-rce": "^17.45.1",
     "@instructure/ui-a11y-content": "^7.20.0",
     "@instructure/ui-alerts": "^7.20.0",
     "@instructure/ui-avatar": "^7.20.0",
@@ -106,7 +106,7 @@
     "eventemitter3": "^3.1.0",
     "humps": "^2.0.0",
     "immutable": "^3.8.1",
-    "instructure-validations": "17.45.1-rc.1+652efed20",
+    "instructure-validations": "^17.45.1",
     "ipaddr.js": "^1.5.4",
     "isomorphic-fetch": "^2.2.0",
     "isuuid": "^0.1.0",
@@ -137,7 +137,7 @@
     "uuid": "^3.2.1"
   },
   "devDependencies": {
-    "@instructure/quiz-scripts": "17.45.0",
+    "@instructure/quiz-scripts": "^17.45.1",
     "@instructure/ui-axe-check": "^7.20.0",
     "@instructure/ui-babel-preset": "^7.20.0",
     "@instructure/ui-karma-config": "^7.20.0",
@@ -156,7 +156,7 @@
     "intl": "^1.2.4",
     "jquery": "^2.2.3",
     "most-subject": "^5.3.0",
-    "quiz-presets": "17.45.1-rc.1+652efed20",
+    "quiz-presets": "^17.45.1",
     "react": "^16.8.6",
     "react-addons-test-utils": "^15.6.2",
     "react-dom": "^16.8.6",
@@ -172,5 +172,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "652efed2087649c08c7ffeb10345e87685b51332"
+  "gitHead": "49b722f452778f3da3ce45a945e87c4427e7c8e7"
 }