Skip to content

I v5 develop 24 aug #4803

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 35 commits into from
Aug 24, 2020
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
35 commits
Select commit Hold shift + click to select a range
490c00b
fix(challenge-listing): sorting functions
cagdas001 Aug 11, 2020
1bf042d
ci: deploy on staging and test env
sushilshinde Aug 13, 2020
52f7a01
Merge pull request #4737 from cagdas001/integration-v5-challenge-api
sushilshinde Aug 13, 2020
75f0c02
fix: testing staging
sushilshinde Aug 13, 2020
5e9ef60
Merge remote-tracking branch 'origin/integration-v5-develop' into i-v…
sushilshinde Aug 17, 2020
03e0bcc
Merge pull request #4767 from topcoder-platform/i-v5-develop-17Aug
sushilshinde Aug 17, 2020
8e1b8e4
Merge pull request #31 from topcoder-platform/integration-v5-challeng…
PrakashDurlabhji Aug 17, 2020
254ae1c
Update index.jsx
PrakashDurlabhji Aug 17, 2020
87d160b
Merge pull request #17 from topcoder-platform/integration-v5-challeng…
rashmi73 Aug 17, 2020
1b32c1f
Update index.jsx
rashmi73 Aug 17, 2020
5cd686e
fix(challenge-listing): sorting by dates & phase displaying
cagdas001 Aug 18, 2020
15a5023
fix(challenge-listing): `CURRENT_PHASE` sorting
cagdas001 Aug 18, 2020
bae2546
Fix Style in Challenge Page
fikzzzy Aug 18, 2020
d67a163
Challenge Details - Fix submissions memberHandle
luizrrodrigues Aug 18, 2020
000c599
ci: synched with integration develop
sushilshinde Aug 18, 2020
dd90b49
ci: sync with integration develop
sushilshinde Aug 18, 2020
1359938
Redirect /challenge/legacyId/submit to /challenge/UUID/submit
luizrrodrigues Aug 19, 2020
cf14434
updated isZurichCompetitor to use group 'oldId' instead 'id'
luizrrodrigues Aug 20, 2020
6a042f5
Merge remote-tracking branch 'origin/develop' into i-v5-develop-20Aug
sushilshinde Aug 20, 2020
d0c866c
Merge pull request #4786 from topcoder-platform/i-v5-develop-20Aug
sushilshinde Aug 20, 2020
d409b61
Merge pull request #4785 from topcoder-platform/issue-4783
sushilshinde Aug 20, 2020
435212a
ci: deploy on dev as well
sushilshinde Aug 20, 2020
1b8d6a7
Merge pull request #4773 from fikzzzy/fix-style-challenge-detail-page
sushilshinde Aug 20, 2020
e318958
Merge pull request #4778 from topcoder-platform/issue-4632
sushilshinde Aug 20, 2020
f5e0e08
Fix Undefined StyleName
fikzzzy Aug 20, 2020
c8b68da
Merge pull request #4790 from fikzzzy/fix-undefined-stylename
sushilshinde Aug 20, 2020
ad04253
Merge pull request #4768 from PrakashDurlabhji/issue_4748
sushilshinde Aug 20, 2020
ea06b4b
Merge pull request #4770 from rashmi73/issue_4746
sushilshinde Aug 20, 2020
436f070
Merge pull request #4772 from cagdas001/fix-4129
sushilshinde Aug 20, 2020
9b07faa
Merge pull request #4774 from topcoder-platform/issue-4667
sushilshinde Aug 20, 2020
f8c7b4c
fix(challenge-listing): fix failing test
cagdas001 Aug 20, 2020
b041baa
Merge pull request #4771 from cagdas001/integration-v5-challenge-api
sushilshinde Aug 21, 2020
38fea9f
fix: null checking
sushilshinde Aug 21, 2020
0f571d8
Merge remote-tracking branch 'origin/develop' into i-v5-develop-24Aug
sushilshinde Aug 24, 2020
a95a319
fix: remove delete me
sushilshinde Aug 24, 2020
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -230,14 +230,14 @@ workflows:
filters:
branches:
only:
- develop
- i-v5-develop-24Aug
# This is alternate dev env for parallel testing
- "build-test":
context : org-global
filters:
branches:
only:
- integration-v5-challenge-api
- hot-fix
# This is alternate dev env for parallel testing
- "build-qa":
context : org-global
Expand All @@ -258,7 +258,7 @@ workflows:
filters:
branches:
only:
- develop
- i-v5-develop-24Aug
# Production builds are exectuted
# when PR is merged to the master
# Don't change anything in this configuration
Expand Down
10 changes: 5 additions & 5 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Original file line number Diff line number Diff line change
Expand Up @@ -36,13 +36,13 @@ export default function SideBar({
const scorecardURL = `${config.URL.ONLINE_REVIEW}/review/actions/ViewScorecard?scid=`;
const faqURL = config.URL.INFO.DESIGN_CHALLENGE_SUBMISSION;
let submissionLimitDisplay = 'Unlimited';
const submissionLimit = _.find(metadata, { type: 'submissionLimit' });
const submissionLimit = _.find(metadata, { name: 'submissionLimit' });
const fileTypes = _.find(metadata, { name: 'fileTypes' });

if (submissionLimit) {
if (submissionLimit.value === 1) {
submissionLimitDisplay = '1 submission';
} else if (submissionLimit > 1) {
} else if (submissionLimit.value > 1) {
submissionLimitDisplay = `${submissionLimit.value} submissions`;
}
}
Expand Down
37 changes: 21 additions & 16 deletions src/shared/components/challenge-detail/Specification/index.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,12 @@ export default function ChallengeDetailsView(props) {
forumId,
} = legacy;

const allowStockArt = _.find(metadata, { type: 'allowStockArt' });
let stockArtValue = '';
const allowStockArt = _.find(metadata, { name: 'allowStockArt' });
if (allowStockArt) {
stockArtValue = allowStockArt.value;
}

let environment = '';
const environmentData = _.find(metadata, { name: 'environment' });
if (environmentData) {
Expand Down Expand Up @@ -103,7 +108,7 @@ export default function ChallengeDetailsView(props) {
const toolbarConnector = new ToolbarConnector();
const isSaving = specsTabState === SPECS_TAB_STATES.SAVING;

const stockArtText = allowStockArt
const stockArtText = stockArtValue
? 'Stock photography is allowed in this challenge.'
: 'Stock photography is not allowed in this challenge. All submitted elements must be designed solely by you.';

Expand Down Expand Up @@ -170,7 +175,7 @@ export default function ChallengeDetailsView(props) {
description
&& (
<article>
<h2 styleName="h2">
<h2>
Challenge Overview
</h2>
{
Expand All @@ -195,7 +200,7 @@ export default function ChallengeDetailsView(props) {
finalSubmissionGuidelines
&& (
<article>
<h2 styleName="h2">
<h2>
Final Submission Guidelines
</h2>
{
Expand Down Expand Up @@ -228,7 +233,7 @@ export default function ChallengeDetailsView(props) {
description
&& (
<article>
<h2 styleName="h2">
<h2>
Challenge Summary
</h2>
{
Expand All @@ -246,8 +251,8 @@ export default function ChallengeDetailsView(props) {
/>
)
}
<p styleName="p" />
<p styleName="p note">
<p />
<p styleName="note">
Please read the challenge specification carefully and
watch the forums for any questions or feedback
concerning this challenge. It is important that you
Expand All @@ -259,10 +264,10 @@ export default function ChallengeDetailsView(props) {
)
}
<article>
<h2 styleName="h2">
<h2>
Stock Photography
</h2>
<p styleName="p">
<p>
{stockArtText}
&nbsp;
<a href={config.URL.INFO.STOCK_ART_POLICY}>
Expand All @@ -271,10 +276,10 @@ export default function ChallengeDetailsView(props) {
</p>
</article>
<article>
<h2 styleName="h2">
<h2>
How To Submit
</h2>
<ul styleName="ul">
<ul>
<li>
New to Studio?
&zwnj;
Expand Down Expand Up @@ -311,10 +316,10 @@ export default function ChallengeDetailsView(props) {
</article>

<article>
<h2 styleName="h2">
<h2>
Winner Selection
</h2>
<p styleName="p">
<p>
Submissions are viewable to the client as they are entered
into the challenge. Winners are selected by the client and
are chosen solely at the client&apos;s discretion.
Expand All @@ -324,13 +329,13 @@ export default function ChallengeDetailsView(props) {
)
}
<article>
<h2 styleName="h2">
<h2>
Payments
</h2>
{
isWipro ? (
<div>
<p styleName="p">
<p>
For employees of Wipro Technologies, following are the
payment terms. Winner/s would be awarded the prize money on
successful completion and acceptance of the submission by
Expand All @@ -354,7 +359,7 @@ export default function ChallengeDetailsView(props) {
</p>
</div>
) : (
<p styleName="p">
<p>
Topcoder will compensate members in accordance with our standard payment policies, unless
otherwise specified in this challenge. For information on payment policies, setting up your profile to
receive payments, and general payment questions, please refer to
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -78,6 +78,8 @@ $tc-link-visited: #0c4e98;
align-items: baseline;
word-break: break-word;

@include linkStyle;

@include xs-to-sm {
flex-direction: column;
}
Expand Down Expand Up @@ -118,7 +120,7 @@ $tc-link-visited: #0c4e98;
}
}

.h2 {
h2 {
@include roboto-bold;

font-size: 20px;
Expand All @@ -127,7 +129,7 @@ $tc-link-visited: #0c4e98;
margin: (6 * $base-unit) 0 (2 * $base-unit);
}

.h3 {
h3 {
@include roboto-bold;

font-size: 15px;
Expand All @@ -137,7 +139,17 @@ $tc-link-visited: #0c4e98;
text-transform: uppercase;
}

.p {
p,
em,
strong {
@include roboto-regular;

font-size: 15px;
color: $tc-gray-90;
line-height: 25px;
}

p {
@include roboto-regular;

font-size: 15px;
Expand All @@ -149,7 +161,7 @@ $tc-link-visited: #0c4e98;
margin: (2 * $base-unit) 0 (3 * $base-unit);
}

.ul {
ul {
@include roboto-regular;

margin: 0 0 5px 20px;
Expand Down
14 changes: 7 additions & 7 deletions src/shared/components/challenge-detail/Submissions/index.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -175,8 +175,8 @@ class SubmissionsComponent extends React.Component {
valueA = `${a.member || ''}`.toLowerCase();
valueB = `${b.member || ''}`.toLowerCase();
} else {
valueA = `${a.createdBy}`.toLowerCase();
valueB = `${b.createdBy}`.toLowerCase();
valueA = _.get(a.registrant, 'memberHandle', '');
valueB = _.get(b.registrant, 'memberHandle', '');
}
valueIsString = true;
break;
Expand Down Expand Up @@ -314,12 +314,12 @@ class SubmissionsComponent extends React.Component {
{`#${s.id}`}
</a>
<a
href={`${window.origin}/members/${s.createdBy}`}
href={`${window.origin}/members/${_.get(s.registrant, 'memberHandle', '')}`}
target={`${_.includes(window.origin, 'www') ? '_self' : '_blank'}`}
rel="noopener noreferrer"
styleName={`level-${getRatingLevel(_.get(s.registrant, 'rating', 0))}`}
>
{s.createdBy}
{_.get(s.registrant, 'memberHandle', '')}
</a>
</div>
<div>
Expand Down Expand Up @@ -705,7 +705,7 @@ class SubmissionsComponent extends React.Component {
{
!isMM && (
sortedSubmissions.map(s => (
<div key={s.createdBy + s.created} styleName="row">
<div key={_.get(s.registrant, 'memberHandle', '') + s.created} styleName="row">
{
!isF2F && !isBugHunt && (
<div styleName={`col-2 level-${getRatingLevel(_.get(s.registrant, 'rating', 0))}`}>
Expand All @@ -715,12 +715,12 @@ class SubmissionsComponent extends React.Component {
}
<div styleName="col-3">
<a
href={`${window.origin}/members/${s.createdBy}`}
href={`${window.origin}/members/${_.get(s.registrant, 'memberHandle', '')}`}
target={`${_.includes(window.origin, 'www') ? '_self' : '_blank'}`}
rel="noopener noreferrer"
styleName={`handle level-${getRatingLevel(_.get(s.registrant, 'rating', 0))}`}
>
{s.createdBy}
{_.get(s.registrant, 'memberHandle', '')}
</a>
</div>
<div styleName="col-4">
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ import LeaderboardAvatar from 'components/challenge-listing/LeaderboardAvatar';
import { config, Link } from 'topcoder-react-utils';
import { TABS as DETAIL_TABS } from 'actions/page/challenge-details';
import 'moment-duration-format';
import { phaseEndDate } from 'utils/challenge-listing/helper';
import {
getTimeLeft,
} from 'utils/challenge-detail/helper';
Expand Down Expand Up @@ -271,7 +272,7 @@ export default function ChallengeStatus(props) {
<ChallengeProgressBar
color="green"
value={getPhaseProgress(statusPhase)}
isLate={moment().isAfter(statusPhase.scheduledEndDate)}
isLate={moment().isAfter(phaseEndDate(statusPhase))}
/>
<div styleName="time-left">
{getTimeLeft(statusPhase, 'to go').text}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ import _ from 'lodash';
import React from 'react';
import PT from 'prop-types';
import Tooltip from 'components/Tooltip';
import { phaseStartDate, phaseEndDate } from 'utils/challenge-listing/helper';
import LoaderIcon from '../../../Loader/Loader';
import './style.scss';

Expand Down Expand Up @@ -95,31 +96,32 @@ function Tip(props) {
if (!c || _.isEmpty(c)) return <div />;

const allPhases = c.phases || [];
const endPhaseDate = Math.max(...allPhases.map(d => new Date(d.scheduledEndDate)));
const endPhaseDate = Math.max(...allPhases.map(d => phaseEndDate(d)));
const registrationPhase = allPhases.find(phase => phase.name === 'Registration');
const submissionPhase = allPhases.find(phase => phase.name === 'Submission');
const checkpointPhase = allPhases.find(phase => phase.name === 'Checkpoint Submission');

if (registrationPhase) {
steps.push({
date: new Date(registrationPhase.scheduledStartDate),
date: phaseStartDate(registrationPhase),
name: 'Start',
});
}
if (c.checkpointSubmissionEndDate) {
if (checkpointPhase) {
steps.push({
date: new Date(c.checkpointSubmissionEndDate),
date: phaseEndDate(checkpointPhase),
name: 'Checkpoint',
});
}
const iterativeReviewPhase = allPhases.find(phase => phase.isOpen && phase.name === 'Iterative Review');
if (iterativeReviewPhase) {
steps.push({
date: new Date(iterativeReviewPhase.scheduledEndDate),
date: phaseEndDate(iterativeReviewPhase),
name: 'Iterative Review',
});
} else if (submissionPhase) {
steps.push({
date: new Date(submissionPhase.scheduledEndDate),
date: phaseEndDate(submissionPhase),
name: 'Submission',
});
}
Expand Down
2 changes: 1 addition & 1 deletion src/shared/components/tc-communities/Header/index.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ function Header(props) {
const AUTH_URL = config.URL.AUTH;
const normalizedProfile = profile && _.clone(profile);
const isZurichCompetitor = (profile && profile.groups) ? _.intersection(
_.map(profile.groups, 'id'),
_.map(profile.groups, 'oldId'),
meta.competitorsGroupIds,
) : [];

Expand Down
Loading