Merge pull request #135 from topcoder-platform/develop-mm-sub-merge-fix

Develop mm sub merge fix for prod release

Author: sushilshinde

Diff for: .circleci/config.yml

@@ -28,8 +28,8 @@ jobs:
       - attach_workspace:
           at: .
       - run: echo "//registry.npmjs.org/:_authToken=$NPM_TOKEN" >> ~/.npmrc
-      - run: npm publish
-
+      - run: npm publish --tag test-release
+# dont change anything
 workflows:
   version: 2
   build:

Diff for: README.md

@@ -10,6 +10,10 @@ The [Topcoder](https://www.topcoder.com) lib for internal ReactJS projects.
 
 Change the property in `AV_SCAN_SCORER_REVIEW_TYPE_ID` in config.
 
+### Configuration for AV-Scan scorer review type ID
+
+Change the property in `PROVISIONAL_SCORING_COMPLETED_REVIEW_TYPE_ID` in config.
+
 ### Development
 ```shell
 # Install dependencies
@@ -19,7 +23,7 @@ npm install
 npm test
 npm run build
 
-# Go to other project which depends on the topcoder-react-lib, config its package.json so 
+# Go to other project which depends on the topcoder-react-lib, config its package.json so
 # that the 'topcoder-react-lib' points to the local foler path of topcoder-react-lib:
 #
 # "dependencies": {

Diff for: __tests__/__snapshots__/index.js.snap

@@ -62,6 +62,8 @@ Object {
       "getAllCountriesInit": [Function],
       "getCountriesDone": [Function],
       "getCountriesInit": [Function],
+      "getReviewTypesDone": [Function],
+      "getReviewTypesInit": [Function],
       "getSkillTagsDone": [Function],
       "getSkillTagsInit": [Function],
     },

Diff for: __tests__/actions/__snapshots__/lookup.js.snap

@@ -7,6 +7,8 @@ Object {
     "getAllCountriesInit": [Function],
     "getCountriesDone": [Function],
     "getCountriesInit": [Function],
+    "getReviewTypesDone": [Function],
+    "getReviewTypesInit": [Function],
     "getSkillTagsDone": [Function],
     "getSkillTagsInit": [Function],
   },

Diff for: __tests__/reducers/__snapshots__/lookup.js.snap

@@ -11,6 +11,7 @@ Object {
   ],
   "loadingCountriesError": false,
   "loadingSkillTagsError": true,
+  "reviewTypes": Array [],
   "skillTags": Array [
     Object {
       "domain": "SKILLS",
@@ -33,6 +34,7 @@ Object {
   ],
   "loadingCountriesError": true,
   "loadingSkillTagsError": true,
+  "reviewTypes": Array [],
   "skillTags": Array [
     Object {
       "domain": "SKILLS",
@@ -49,6 +51,7 @@ Object {
   "allCountries": Array [],
   "countries": Array [],
   "loadingSkillTagsError": false,
+  "reviewTypes": Array [],
   "skillTags": Array [
     Object {
       "domain": "SKILLS",
@@ -65,6 +68,7 @@ Object {
   "allCountries": Array [],
   "countries": Array [],
   "loadingSkillTagsError": true,
+  "reviewTypes": Array [],
   "skillTags": Array [
     Object {
       "domain": "SKILLS",
@@ -80,6 +84,7 @@ exports[`Default reducer Initial state 1`] = `
 Object {
   "allCountries": Array [],
   "countries": Array [],
+  "reviewTypes": Array [],
   "skillTags": Array [],
 }
 `;
@@ -95,6 +100,7 @@ Object {
   ],
   "loadingCountriesError": false,
   "loadingSkillTagsError": true,
+  "reviewTypes": Array [],
   "skillTags": Array [
     Object {
       "domain": "SKILLS",
@@ -117,6 +123,7 @@ Object {
   ],
   "loadingCountriesError": true,
   "loadingSkillTagsError": true,
+  "reviewTypes": Array [],
   "skillTags": Array [
     Object {
       "domain": "SKILLS",
@@ -133,6 +140,7 @@ Object {
   "allCountries": Array [],
   "countries": Array [],
   "loadingSkillTagsError": false,
+  "reviewTypes": Array [],
   "skillTags": Array [
     Object {
       "domain": "SKILLS",
@@ -149,6 +157,7 @@ Object {
   "allCountries": Array [],
   "countries": Array [],
   "loadingSkillTagsError": true,
+  "reviewTypes": Array [],
   "skillTags": Array [
     Object {
       "domain": "SKILLS",
@@ -164,6 +173,7 @@ exports[`Factory without server side rendering Initial state 1`] = `
 Object {
   "allCountries": Array [],
   "countries": Array [],
+  "reviewTypes": Array [],
   "skillTags": Array [],
 }
 `;

Diff for: config/default.json

@@ -1,5 +1,6 @@
 {
   "AV_SCAN_SCORER_REVIEW_TYPE_ID": "",
+  "PROVISIONAL_SCORING_COMPLETED_REVIEW_TYPE_ID": "",
   "PAGE_SIZE": 50,
   "REVIEW_OPPORTUNITY_PAGE_SIZE": 1000
 }

Diff for: config/development.json

@@ -1,3 +1,4 @@
 {
-  "AV_SCAN_SCORER_REVIEW_TYPE_ID": "68c5a381-c8ab-48af-92a7-7a869a4ee6c3"
+  "AV_SCAN_SCORER_REVIEW_TYPE_ID": "68c5a381-c8ab-48af-92a7-7a869a4ee6c3",
+  "PROVISIONAL_SCORING_COMPLETED_REVIEW_TYPE_ID": "52c91e85-745f-4e62-b592-9879a2dfe9fd"
 }

Diff for: config/production.json

@@ -1,3 +1,4 @@
 {
-  "AV_SCAN_SCORER_REVIEW_TYPE_ID": "55bbb17d-aac2-45a6-89c3-a8d102863d05"
+  "AV_SCAN_SCORER_REVIEW_TYPE_ID": "55bbb17d-aac2-45a6-89c3-a8d102863d05",
+  "PROVISIONAL_SCORING_COMPLETED_REVIEW_TYPE_ID": "df51ca7d-fb0a-4147-9569-992fcf5aae48"
 }

Diff for: config/test.js

@@ -4,4 +4,12 @@ module.exports = {
     V3: 'https://api.topcoder-dev.com/v3',
   },
   dummyConfigKey: 'Dummy config value',
+  SECRET: {
+    TC_M2M: {
+      AUTH0_URL: '',
+      AUTH0_AUDIENCE: '',
+      TOKEN_CACHE_TIME: '',
+      AUTH0_PROXY_SERVER_URL: '',
+    },
+  },
 };

Diff for: config/webpack/default.js

@@ -2,10 +2,11 @@
 const webpack = require('webpack');
 
 module.exports = {
-  plugins: [
-    // eslint-disable-next-line global-require
-    new webpack.DefinePlugin({ CONFIG: JSON.stringify(require('config')) }),
-  ],
+  node: {
+    tls: 'empty',
+    fs: 'empty',
+    net: 'empty',
+  },
   // Don't include the dependencies to keep built bundle small,
   // they will be provided by the app using this lib
   externals: [
@@ -24,8 +25,10 @@ module.exports = {
     'tc-accounts',
     'to-capital-case',
     'topcoder-react-utils',
+    'tc-core-library-js',
+  ],
+  plugins: [
+    // eslint-disable-next-line global-require
+    new webpack.DefinePlugin({ CONFIG: JSON.stringify(require('config')) }),
   ],
-  node: {
-    fs: 'empty',
-  },
 };

Diff for: dist/dev/index.js

@@ -11,6 +11,8 @@ Actions related to lookup data.
     * [.getCountriesDone()](#module_actions.lookup.getCountriesDone) ⇒ <code>Action</code>
     * [.getAllCountriesInit()](#module_actions.lookup.getCountriesInit) ⇒ <code>Action</code>
     * [.getAllCountriesDone(tokenV3)](#module_actions.lookup.getCountriesDone) ⇒ <code>Action</code>
+    * [.getReviewTypesInit()](#module_actions.lookup.getReviewTypesInit) ⇒ <code>Action</code>
+    * [.getReviewTypesDone(tokenV3)](#module_actions.lookup.ggetReviewTypesDone) ⇒ <code>Action</code>
 
 <a name="module_actions.lookup.getSkillTagsInit"></a>
 
@@ -40,3 +42,19 @@ Creates an action that gets all countries new version.
 | Param | Type | Description |
 | --- | --- | --- |
 | tokenV3 | <code>String</code> | Topcoder v3 auth token. |
+<a name="module_actions.lookup.getReviewTypesInit"></a>
+
+### actions.lookup.getReviewTypesInit() ⇒ <code>Action</code>
+Creates an action that gets all reiview types.
+
+**Kind**: static method of [<code>actions.lookup</code>](#module_actions.lookup)
+<a name="module_actions.lookup.getReviewTypesDone"></a>
+
+### actions.lookup.getReviewTypesDone(tokenV3) ⇒ <code>Action</code>
+Creates an action that gets all review types.
+
+**Kind**: static method of [<code>actions.lookup</code>](#module_actions.lookup)
+
+| Param | Type | Description |
+| --- | --- | --- |
+| tokenV3 | <code>String</code> | Topcoder v3 auth token. |

Diff for: docs/actions.lookup.md

@@ -18,35 +18,48 @@ State segment managed by this reducer has the following structure:
     * _inner_
         * [~onGetSkillTagsDone(state, action)](#module_reducers.lookup..onGetSkillTagsDone) ⇒ <code>Object</code>
         * [~onGetCountriesDone(state, action)](#module_reducers.lookup..onGetCountriesDone) ⇒ <code>Object</code>
+        * [~onGetReviewTypesDone(state, action)](#module_reducers.lookup..onGetReviewTypesDone) ⇒ <code>Object</code>
         * [~create(initialState)](#module_reducers.lookup..create) ⇒ <code>function</code>
 
 <a name="module_reducers.lookup.default"></a>
 
 ### reducers.lookup.default
 Reducer with default initial state.
 
-**Kind**: static property of [<code>reducers.lookup</code>](#module_reducers.lookup)  
+**Kind**: static property of [<code>reducers.lookup</code>](#module_reducers.lookup)
 <a name="module_reducers.lookup.factory"></a>
 
 ### reducers.lookup.factory() ⇒ <code>Promise</code>
 Factory which creates a new reducer.
 
-**Kind**: static method of [<code>reducers.lookup</code>](#module_reducers.lookup)  
-**Resolves**: <code>Function(state, action): state</code> New reducer.  
+**Kind**: static method of [<code>reducers.lookup</code>](#module_reducers.lookup)
+**Resolves**: <code>Function(state, action): state</code> New reducer.
 <a name="module_reducers.lookup..onGetSkillTagsDone"></a>
 
 ### reducers.lookup~onGetSkillTagsDone(state, action) ⇒ <code>Object</code>
 Handles LOOKUP/GET_SKILL_TAGS_DONE action.
 
-**Kind**: static method of [<code>reducers.lookup</code>](#module_reducers.lookup)  
-**Resolves**: <code>Function(state, action): state</code> New reducer.  
+**Kind**: static method of [<code>reducers.lookup</code>](#module_reducers.lookup)
+**Resolves**: <code>Function(state, action): state</code> New reducer.
 <a name="module_reducers.lookup..onGetCountriesDone"></a>
 
 ### reducers.lookup~onGetCountriesDone(state, action) ⇒ <code>Object</code>
 Handles LOOKUP/GET_COUNTRIES_DONE action.
 
-**Kind**: inner method of [<code>reducers.lookup</code>](#module_reducers.lookup)  
-**Returns**: <code>Object</code> - New state  
+**Kind**: inner method of [<code>reducers.lookup</code>](#module_reducers.lookup)
+**Returns**: <code>Object</code> - New state
+
+| Param | Type | Description |
+| --- | --- | --- |
+| state | <code>Object</code> |  |
+| action | <code>Object</code> | Payload will be JSON from api call |
+<a name="module_reducers.lookup..onGetReviewTypesDone"></a>
+
+### reducers.lookup~onGetReviewTypesDone(state, action) ⇒ <code>Object</code>
+Handles LOOKUP/GET_REVIEW_TYPES_DONE action.
+
+**Kind**: inner method of [<code>reducers.lookup</code>](#module_reducers.lookup)
+**Returns**: <code>Object</code> - New state
 
 | Param | Type | Description |
 | --- | --- | --- |
@@ -58,8 +71,8 @@ Handles LOOKUP/GET_COUNTRIES_DONE action.
 ### reducers.lookup~create(initialState) ⇒ <code>function</code>
 Creates a new Lookup reducer with the specified initial state.
 
-**Kind**: inner method of [<code>reducers.lookup</code>](#module_reducers.lookup)  
-**Returns**: <code>function</code> - Lookup reducer.  
+**Kind**: inner method of [<code>reducers.lookup</code>](#module_reducers.lookup)
+**Returns**: <code>function</code> - Lookup reducer.
 
 | Param | Type | Description |
 | --- | --- | --- |

Diff for: docs/reducers.lookup.md

@@ -13,6 +13,7 @@ via API V5.
             * [new SubmissionsService(tokenV3)](#new_module_services.submissions..SubmissionsService_new)
             * [.getSubmissions(filters, params)](#module_services.submissions..SubmissionsService+getSubmissions) ⇒ <code>Promise</code>
             * [.getSubmissionInformation(submissionId)](#module_services.submissions..SubmissionsService+getSubmissionInformation) ⇒ <code>Promise</code>
+            * [.downloadSubmission(submissionId)](#module_services.submissions..SubmissionsService+downloadSubmission) ⇒ <code>Promise</code>
 
 <a name="module_services.submissions.getService"></a>
 
@@ -35,6 +36,7 @@ Returns a new or existing submissions service.
     * [new SubmissionsService(tokenV3)](#new_module_services.submissions..SubmissionsService_new)
     * [.getSubmissions(filters, params)](#module_services.submissions..SubmissionsService+getSubmissions) ⇒ <code>Promise</code>
     * [.getSubmissionInformation(submissionId)](#module_services.submissions..SubmissionsService+getSubmissionInformation) ⇒ <code>Promise</code>
+    * [.downloadSubmission(submissionId)](#module_services.submissions..SubmissionsService+downloadSubmission) ⇒ <code>Promise</code>
 
 <a name="new_module_services.submissions..SubmissionsService_new"></a>
 
@@ -68,3 +70,15 @@ Gets submission information.
 | Param | Type | Description |
 | --- | --- | --- |
 | submissionId | <code>String</code> | The id of submission |
+
+<a name="module_services.submissions..SubmissionsService+downloadSubmission"></a>
+
+#### submissionsService.downloadSubmission(submissionId) ⇒ <code>Promise</code>
+Download submission.
+
+**Kind**: instance method of [<code>SubmissionsService</code>](#module_services.submissions..SubmissionsService)
+**Returns**: <code>Promise</code> - Resolves to the submission file.
+
+| Param | Type | Description |
+| --- | --- | --- |
+| submissionId | <code>String</code> | The id of submission |