Skip to content

Commit 9af2d08

Browse files
authored
Merge pull request #708 from topcoder-platform/dev
[PROD] TSJR-365
2 parents 4ee9405 + 726e7fe commit 9af2d08

15 files changed

+32
-77
lines changed

.circleci/config.yml

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,15 @@
11
version: 2
22
defaults: &defaults
33
docker:
4-
- image: cimg/python:3.11.0-browsers
4+
- image: cimg/python:3.12.1-browsers
55
install_dependency: &install_dependency
66
name: Installation of build and deployment dependencies.
77
command: |
8-
sudo apt update
9-
sudo apt install jq
10-
sudo apt install python3-pip
11-
sudo pip3 install awscli --upgrade
12-
sudo pip3 install docker-compose
8+
pip3 install awscli --upgrade
139
install_deploysuite: &install_deploysuite
1410
name: Installation of install_deploysuite.
1511
command: |
16-
git clone --branch v1.4.15 https://github.com/topcoder-platform/tc-deploy-scripts ../buildscript
12+
git clone --branch v1.4.17 https://github.com/topcoder-platform/tc-deploy-scripts ../buildscript
1713
cp ./../buildscript/master_deploy.sh .
1814
cp ./../buildscript/buildenv.sh .
1915
cp ./../buildscript/awsconfiguration.sh .
@@ -90,7 +86,6 @@ workflows:
9086
branches:
9187
only:
9288
- dev
93-
- CORE-40
9489

9590
- "build-qa":
9691
context: org-global

.nvmrc

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
18.19.0

README.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -105,6 +105,10 @@ You can find sample `.env` files inside the `/docs` directory.
105105
1. 📦 Install npm dependencies
106106

107107
```bash
108+
# export the production AWS credentials to access the topcoder-framework private repos in AWS codeartifact
109+
aws codeartifact login --tool npm --repository topcoder-framework --domain topcoder --domain-owner 409275337247 --region us-east-1 --namespace @topcoder-framework
110+
111+
# install dependencies
108112
yarn install
109113
```
110114

src/common/challenge-helper.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -377,9 +377,9 @@ class ChallengeHelper {
377377
if (data.prizeSets != null) {
378378
const type = data.prizeSets[0]?.prizes[0]?.type;
379379
if (type === constants.prizeTypes.USD) {
380-
ChallengeHelper.convertPSValuesToCents(data.prizeSets)
380+
ChallengeHelper.convertPSValuesToCents(data.prizeSets);
381381
}
382-
382+
383383
data.prizeSetUpdate = {
384384
prizeSets: [...data.prizeSets],
385385
};
@@ -489,7 +489,7 @@ class ChallengeHelper {
489489
}
490490
}
491491

492-
static convertPSValuesToCents(prizeSets){
492+
static convertPSValuesToCents(prizeSets) {
493493
prizeSets.forEach((prizeSet) => {
494494
prizeSet.prizes.forEach((prize) => {
495495
prize.amountInCents = new Decimal(prize.value).mul(100).toNumber();

src/common/helper.js

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1139,7 +1139,6 @@ async function getMembersByHandles(handles) {
11391139
* @returns {Object}
11401140
*/
11411141
async function getStandSkills(ids) {
1142-
11431142
const queryBatches = [];
11441143
const skillIdArg = "&skillId=";
11451144
let queryString = "disablePagination=true";
@@ -1157,15 +1156,11 @@ async function getStandSkills(ids) {
11571156
queryBatches.push(queryString);
11581157

11591158
const skillDataPromises = [];
1160-
const token = await m2mHelper.getM2MToken();
11611159
for (const batch of queryBatches) {
11621160
skillDataPromises.push(
11631161
(async () => {
11641162
const res = await axios.get(
1165-
`${config.API_BASE_URL}/v5/standardized-skills/skills?${batch}`,
1166-
{
1167-
headers: { Authorization: `Bearer ${token}` },
1168-
}
1163+
`${config.API_BASE_URL}/v5/standardized-skills/skills?${batch}`
11691164
);
11701165
return res.data;
11711166
})()

src/common/s3ParseUrl.js

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -17,9 +17,7 @@ module.exports = function s3ParseUrl(url) {
1717
}
1818

1919
// http://s3-aws-region.amazonaws.com/bucket/key1/key2
20-
match = decodedUrl.match(
21-
/^https?:\/\/s3-([^.]+).amazonaws.com\/([^/]+)\/?(.*?)$/
22-
);
20+
match = decodedUrl.match(/^https?:\/\/s3-([^.]+).amazonaws.com\/([^/]+)\/?(.*?)$/);
2321
if (match) {
2422
return {
2523
bucket: match[2],
@@ -40,9 +38,7 @@ module.exports = function s3ParseUrl(url) {
4038

4139
// http://bucket.s3-aws-region.amazonaws.com/key1/key2 or,
4240
// http://bucket.s3.aws-region.amazonaws.com/key1/key2
43-
match = decodedUrl.match(
44-
/^https?:\/\/([^.]+).(?:s3-|s3\.)([^.]+).amazonaws.com\/?(.*?)$/
45-
);
41+
match = decodedUrl.match(/^https?:\/\/([^.]+).(?:s3-|s3\.)([^.]+).amazonaws.com\/?(.*?)$/);
4642
if (match) {
4743
return {
4844
bucket: match[1],

src/controllers/AttachmentController.js

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12,11 +12,7 @@ const service = require("../services/AttachmentService");
1212
*/
1313
async function createAttachment(req, res) {
1414
const body = _.isArray(req.body) ? req.body : [req.body];
15-
const result = await service.createAttachment(
16-
req.authUser,
17-
req.params.challengeId,
18-
body
19-
);
15+
const result = await service.createAttachment(req.authUser, req.params.challengeId, body);
2016
res.status(HttpStatus.CREATED).send(result);
2117
}
2218

src/controllers/ChallengePhaseController.js

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -42,10 +42,7 @@ async function getPhase(req, res) {
4242
* @param {Object} res the response
4343
*/
4444
async function fullyUpdatePhase(req, res) {
45-
const result = await service.fullyUpdatePhase(
46-
req.params.challengePhaseId,
47-
req.body
48-
);
45+
const result = await service.fullyUpdatePhase(req.params.challengePhaseId, req.body);
4946
res.send(result);
5047
}
5148

@@ -55,10 +52,7 @@ async function fullyUpdatePhase(req, res) {
5552
* @param {Object} res the response
5653
*/
5754
async function partiallyUpdatePhase(req, res) {
58-
const result = await service.partiallyUpdatePhase(
59-
req.params.challengePhaseId,
60-
req.body
61-
);
55+
const result = await service.partiallyUpdatePhase(req.params.challengePhaseId, req.body);
6256
res.send(result);
6357
}
6458

src/controllers/ChallengeTimelineTemplateController.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,9 +32,7 @@ async function createChallengeTimelineTemplate(req, res) {
3232
* @param {Object} res the response
3333
*/
3434
async function getChallengeTimelineTemplate(req, res) {
35-
const result = await service.getChallengeTimelineTemplate(
36-
req.params.challengeTimelineTemplateId
37-
);
35+
const result = await service.getChallengeTimelineTemplate(req.params.challengeTimelineTemplateId);
3836
res.send(result);
3937
}
4038

src/controllers/HealthController.js

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -23,14 +23,9 @@ async function checkHealth(req, res) {
2323
try {
2424
await service.searchChallengeTypes({ page: 1, perPage: 1 });
2525
} catch (e) {
26-
throw new errors.ServiceUnavailableError(
27-
`There is database operation error, ${e.message}`
28-
);
26+
throw new errors.ServiceUnavailableError(`There is database operation error, ${e.message}`);
2927
}
30-
if (
31-
new Date().getTime() - timestampMS >
32-
Number(config.HEALTH_CHECK_TIMEOUT)
33-
) {
28+
if (new Date().getTime() - timestampMS > Number(config.HEALTH_CHECK_TIMEOUT)) {
3429
throw new errors.ServiceUnavailableError("Database operation is slow.");
3530
}
3631
// there is no error, and it is quick, then return checks run count

src/controllers/SupportController.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,9 +12,7 @@ const logger = require("../common/logger");
1212
*/
1313
async function createRequest(req, res) {
1414
logger.debug(
15-
`createRequest User: ${JSON.stringify(
16-
req.authUser
17-
)} - Body: ${JSON.stringify(req.body)}`
15+
`createRequest User: ${JSON.stringify(req.authUser)} - Body: ${JSON.stringify(req.body)}`
1816
);
1917
const result = await service.createRequest(req.authUser, req.body);
2018
res.status(HttpStatus.CREATED).send(result);

src/controllers/TimelineTemplateController.js

Lines changed: 3 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -32,9 +32,7 @@ async function createTimelineTemplate(req, res) {
3232
* @param {Object} res the response
3333
*/
3434
async function getTimelineTemplate(req, res) {
35-
const result = await service.getTimelineTemplate(
36-
req.params.timelineTemplateId
37-
);
35+
const result = await service.getTimelineTemplate(req.params.timelineTemplateId);
3836
res.send(result);
3937
}
4038

@@ -44,10 +42,7 @@ async function getTimelineTemplate(req, res) {
4442
* @param {Object} res the response
4543
*/
4644
async function fullyUpdateTimelineTemplate(req, res) {
47-
const result = await service.fullyUpdateTimelineTemplate(
48-
req.params.timelineTemplateId,
49-
req.body
50-
);
45+
const result = await service.fullyUpdateTimelineTemplate(req.params.timelineTemplateId, req.body);
5146
res.send(result);
5247
}
5348

@@ -70,9 +65,7 @@ async function partiallyUpdateTimelineTemplate(req, res) {
7065
* @param {Object} res the response
7166
*/
7267
async function deleteTimelineTemplate(req, res) {
73-
const result = await service.deleteTimelineTemplate(
74-
req.params.timelineTemplateId
75-
);
68+
const result = await service.deleteTimelineTemplate(req.params.timelineTemplateId);
7669
res.send(result);
7770
}
7871

src/scripts/check-templates.js

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,11 @@ async function main() {
1111
const phases = res.data;
1212
_.each(template.phases, (phase) => {
1313
const phaseInstance = _.find(phases, (p) => p.id === phase.phaseId);
14-
const pred = phase.predecessor
15-
? _.find(phases, (p) => p.id === phase.predecessor)
16-
: null;
14+
const pred = phase.predecessor ? _.find(phases, (p) => p.id === phase.predecessor) : null;
1715
console.log(
18-
`Phase Length: ${phase.defaultDuration / 60 / 60} hrs \t ${
19-
phaseInstance.name
20-
} - Depends on ${pred ? pred.name : "nothing"}`
16+
`Phase Length: ${phase.defaultDuration / 60 / 60} hrs \t ${phaseInstance.name} - Depends on ${
17+
pred ? pred.name : "nothing"
18+
}`
2119
);
2220
});
2321
}

src/scripts/seed-tables.js

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -18,9 +18,7 @@ Object.keys(models).forEach((modelName) => {
1818
}
1919
try {
2020
const data = require(`./seed/${modelName}.json`);
21-
logger.info(
22-
`Inserting ${get(data, "length")} records in table ${modelName}`
23-
);
21+
logger.info(`Inserting ${get(data, "length")} records in table ${modelName}`);
2422
promises.push(models[modelName].batchPut(data));
2523
} catch (e) {
2624
logger.warn(`No records will be inserted in table ${modelName}`);
@@ -29,9 +27,7 @@ Object.keys(models).forEach((modelName) => {
2927

3028
Promise.all(promises)
3129
.then(() => {
32-
logger.info(
33-
"All tables have been inserted with the data. The processes is run asynchronously"
34-
);
30+
logger.info("All tables have been inserted with the data. The processes is run asynchronously");
3531
process.exit();
3632
})
3733
.catch((err) => {

src/scripts/view-data.js

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,7 @@ const viewData = async (modelName) => {
1515
};
1616

1717
if (process.argv.length === 2) {
18-
logger.info(
19-
`Please provide one of the following table name: [${_.keys(models)}]`
20-
);
18+
logger.info(`Please provide one of the following table name: [${_.keys(models)}]`);
2119
process.exit(1);
2220
} else {
2321
const modelName = process.argv[2];
@@ -32,9 +30,7 @@ if (process.argv.length === 2) {
3230
process.exit(1);
3331
});
3432
} else {
35-
logger.info(
36-
`Please provide one of the following table name: [${_.keys(models)}]`
37-
);
33+
logger.info(`Please provide one of the following table name: [${_.keys(models)}]`);
3834
process.exit(1);
3935
}
4036
}

0 commit comments

Comments
 (0)