Skip to content

Commit 023db8b

Browse files
luizrrodriguessushilshinde
authored andcommittedSep 1, 2020
Merge pull request #238 from topcoder-platform/issue-4739-hotfix
Fix Review Opportunities QA filter
1 parent 25e166b commit 023db8b

File tree

3 files changed

+50
-0
lines changed

3 files changed

+50
-0
lines changed
 

‎__tests__/__snapshots__/index.js.snap

+15
Original file line numberDiff line numberDiff line change
@@ -377,6 +377,21 @@ Object {
377377
"DEVELOP": "Development",
378378
"QA": "Quality Assurance",
379379
},
380+
<<<<<<< HEAD
381+
=======
382+
"OLD_COMPETITION_TRACKS": Object {
383+
"DATA_SCIENCE": "DATA_SCIENCE",
384+
"DESIGN": "DESIGN",
385+
"DEVELOP": "DEVELOP",
386+
"QA": "QA",
387+
},
388+
"OLD_SUBTRACKS": Object {
389+
"BUG_HUNT": "BUG_HUNT",
390+
"TESTING_COMPETITION": "TESTING_COMPETITION",
391+
"TEST_SCENARIOS": "TEST_SCENARIOS",
392+
"TEST_SUITES": "TEST_SUITES",
393+
},
394+
>>>>>>> 978da83... Merge pull request #238 from topcoder-platform/issue-4739-hotfix
380395
"REVIEW_OPPORTUNITY_TYPES": Object {
381396
"Contest Review": "Review",
382397
"Iterative Review": "Iterative Review",

‎src/services/reviewOpportunities.js

+21
Original file line numberDiff line numberDiff line change
@@ -10,12 +10,33 @@ import { getApi } from './api';
1010
* Sync the fields of V3 and V5 for front-end to process successfully
1111
* @param challenges - challenges to normalize
1212
*/
13+
<<<<<<< HEAD
1314
export function normalizeChallenges(challenges) {
1415
if (challenges) {
1516
_.map(challenges, (ch) => {
1617
const { challenge } = ch;
1718
if (challenge.technologies && challenge.technologies.includes('Data Science')) {
1819
challenge.track = 'DATA_SCIENCE';
20+
=======
21+
export function normalizeChallenges(opportunities) {
22+
if (opportunities) {
23+
/* Issue#4739 : Temporary add track to review opportunities challenges
24+
* until receive API V5 update. */
25+
_.map(opportunities, (opportunity) => {
26+
const { challenge } = opportunity;
27+
challenge.track = COMPETITION_TRACKS.DEVELOP;
28+
if (challenge.technologies) {
29+
if (challenge.technologies.includes(COMPETITION_TRACKS.DATA_SCIENCE)) {
30+
challenge.track = COMPETITION_TRACKS.DATA_SCIENCE;
31+
}
32+
} else if (challenge.subTrack === OLD_SUBTRACKS.TEST_SUITES
33+
|| challenge.subTrack === OLD_SUBTRACKS.BUG_HUNT
34+
|| challenge.subTrack === OLD_COMPETITION_TRACKS.TEST_SCENARIOS
35+
|| challenge.subTrack === OLD_COMPETITION_TRACKS.TESTING_COMPETITION) {
36+
challenge.track = COMPETITION_TRACKS.QA;
37+
} else if (challenge.track === OLD_COMPETITION_TRACKS.DESIGN) {
38+
challenge.track = COMPETITION_TRACKS.DESIGN;
39+
>>>>>>> 978da83... Merge pull request #238 from topcoder-platform/issue-4739-hotfix
1940
}
2041
return _.defaults(ch, { challenge });
2142
});

‎src/utils/tc.js

+14
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,20 @@ export const COMPETITION_TRACKS = {
1717
QA: 'Quality Assurance',
1818
};
1919

20+
export const OLD_COMPETITION_TRACKS = {
21+
DATA_SCIENCE: 'DATA_SCIENCE',
22+
DESIGN: 'DESIGN',
23+
DEVELOP: 'DEVELOP',
24+
QA: 'QA',
25+
};
26+
27+
export const OLD_SUBTRACKS = {
28+
TEST_SUITES: 'TEST_SUITES',
29+
BUG_HUNT: 'BUG_HUNT',
30+
TEST_SCENARIOS: 'TEST_SCENARIOS',
31+
TESTING_COMPETITION: 'TESTING_COMPETITION',
32+
};
33+
2034
/**
2135
* Review Opportunity types
2236
*/

0 commit comments

Comments
 (0)
Please sign in to comment.