Merge pull request #6844 from topcoder-platform/revert-6841-develop

Revert "Revert RDM submission download changes for further QA --> Prod"
topcoder-platform · Mar 8, 2023 · 9e0f88e · 9e0f88e
2 parents 6a8263e + 0d487ab
commit 9e0f88e
Show file tree

Hide file tree

Showing 17 changed files with 76 additions and 77 deletions.
diff --git a/.eslintrc b/.eslintrc
@@ -4,7 +4,10 @@
     "jsx-a11y/anchor-is-valid": false,
     "import/no-cycle": [2, { "maxDepth": 1 }],
     "react/forbid-prop-types": false,
-    "react/no-unknown-property": ["error", { "ignore": ["styleName"] }]
+    "react/no-unknown-property": false,
+    "react/jsx-no-bind": false,
+    "react/destructuring-assignment": false,
+    "react/no-array-index-key": false
   },
   "env": {
     "browser": true

diff --git a/src/shared/components/Contentful/AppComponent/index.jsx b/src/shared/components/Contentful/AppComponent/index.jsx
@@ -18,59 +18,41 @@ import PathSelector from 'components/MemberPath/PathSelector';
 const { fireErrorMessage } = errors;
 
 export function AppComponentSwitch(appComponent) {
-  const {
-    fields: {
-      type,
-      theme,
-      props,
-      props: {
-        leaderboardApiUrl,
-        title,
-        podiumSpots,
-        isCopilot,
-        hasChallengeHistory,
-        tcoPointsApiUrl,
-        memberLimit,
-        isAlgo,
-      },
-    },
-    sys: { id },
-  } = appComponent;
-  if (type === 'TCO-Leaderboard') {
+  if (appComponent.fields.type === 'TCO-Leaderboard') {
     return (
       <Leaderboard
-        id={id}
-        apiUrl={leaderboardApiUrl}
-        title={title}
-        podiumSpots={podiumSpots}
-        isCopilot={isCopilot}
-        hasChallengeHistory={hasChallengeHistory}
-        tcoPointsApiUrl={tcoPointsApiUrl}
-        memberLimit={memberLimit}
-        isAlgo={isAlgo}
-        key={id}
-        themeName={theme}
+        id={appComponent.sys.id}
+        apiUrl={appComponent.fields.props.leaderboardApiUrl}
+        title={appComponent.fields.props.title}
+        podiumSpots={appComponent.fields.props.podiumSpots}
+        isCopilot={appComponent.fields.props.isCopilot}
+        hasChallengeHistory={appComponent.fields.props.hasChallengeHistory}
+        tcoPointsApiUrl={appComponent.fields.props.tcoPointsApiUrl}
+        memberLimit={appComponent.fields.props.memberLimit}
+        isAlgo={appComponent.fields.props.isAlgo}
+        key={appComponent.sys.id}
+        themeName={appComponent.fields.theme}
       />
     );
   }
-  if (type === 'RecruitCRM-Jobs') {
-    return <RecruitCRMJobs {...props} key={id} />;
+  if (appComponent.fields.type === 'RecruitCRM-Jobs') {
+    return <RecruitCRMJobs {...appComponent.fields.props} key={appComponent.sys.id} />;
   }
-  if (type === 'EmailSubscribeForm') {
-    return <EmailSubscribeForm {...props} key={id} />;
+  if (appComponent.fields.type === 'EmailSubscribeForm') {
+    return <EmailSubscribeForm {...appComponent.fields.props} key={appComponent.sys.id} />;
   }
-  if (type === 'GSheet') {
-    return <GSheet {...props} key={id} />;
+  if (appComponent.fields.type === 'GSheet') {
+    return <GSheet {...appComponent.fields.props} key={appComponent.sys.id} />;
   }
-  if (type === 'MemberPath') {
+  if (appComponent.fields.type === 'MemberPath') {
     return (
       <PathSelector
-        {...props}
-        key={id}
+        {...appComponent.fields.props}
+        key={appComponent.sys.id}
       />
     );
   }
-  fireErrorMessage(`Unsupported app component type ${type}`, '');
+  fireErrorMessage(`Unsupported app component type ${appComponent.fields.type}`, '');
   return null;
 }
 

diff --git a/src/shared/components/Contentful/Viewport/index.jsx b/src/shared/components/Contentful/Viewport/index.jsx
@@ -272,7 +272,7 @@ export function ViewportLoader(props) {
   }
 
   if (query) {
-    queries.push({ ...query, content_type: 'viewport' });
+    queries.push({ ...props.query, content_type: 'viewport' });
   }
 
   return (

diff --git a/src/shared/components/Gigs/ReferralCode/index.jsx b/src/shared/components/Gigs/ReferralCode/index.jsx
@@ -25,7 +25,7 @@ function ReferralCode(props) {
   const [growSurfState, setGrowSurfState] = useState(growSurf);
   const [copyBtnText, setCopyBtnText] = useState('COPY');
   useEffect(() => {
-    setGrowSurfState(growSurf);
+    setGrowSurfState(props.growSurf);
   }, [growSurf]);
 
   return (

diff --git a/src/shared/components/Settings/Account/LinkedAccount/ExistingLink.jsx b/src/shared/components/Settings/Account/LinkedAccount/ExistingLink.jsx
@@ -64,15 +64,7 @@ export default function ExistingLink(props) {
   return (
     <div role="link" styleName={`external-link-tile ${pending ? 'external-link-tile--pending' : ''}`} onClick={e => openLink(e, link)}>
       <div styleName="ext-link-tile_edit-header">
-        <div
-          role="button"
-          onClick={(e) => {
-            e.preventDefault();
-            e.stopPropagation();
-            onConfirmDeleteLink(e, link);
-          }}
-          styleName={`ext-link-tile_edit-header_delete ${link.deleting ? 'ext-link-tile_edit-header_delete--disabled' : ''}`}
-        />
+        <div role="button" onClick={e => onConfirmDeleteLink(e, link)} styleName={`ext-link-tile_edit-header_delete ${link.deleting ? 'ext-link-tile_edit-header_delete--disabled' : ''}`} prevent-event-propagation="true" />
       </div>
       <div styleName="top">
         <div styleName="logo">
@@ -102,7 +94,7 @@ export default function ExistingLink(props) {
               <p className={!pending ? 'hidden' : ''} styleName="link-title">
                 Loading data. This will take a few minutes.
               </p>
-              <a styleName="link-url" id="link-url" href={prependProtocol(link.URL)} target="_blank" rel="noopener noreferrer">
+              <a styleName="link-url" id="link-url" href={prependProtocol(link.URL)} target="_blank" rel="noopener noreferrer" prevent-event-propagation="true">
                 {prependProtocol(link.URL)}
               </a>
             </div>

diff --git a/src/shared/components/challenge-detail/Header/TabSelector/index.jsx b/src/shared/components/challenge-detail/Header/TabSelector/index.jsx
@@ -404,9 +404,9 @@ export default function ChallengeViewSelector(props) {
                   </div>
                   <div styleName="sort-body">
                     {
-                      sortOptions.map(option => (
+                      sortOptions.map((option, index) => (
                         <div
-                          key={`sort-option-${option.name}`}
+                          map={`sort-option-${index}`}
                           styleName="sort-item"
                           onClick={() => {
                             setSelectedSortOption(option.name);

diff --git a/...ared/components/challenge-detail/Submissions/SubmissionRow/SubmissionHistoryRow/index.jsx b/...ared/components/challenge-detail/Submissions/SubmissionRow/SubmissionHistoryRow/index.jsx
@@ -21,6 +21,7 @@ const { getService } = services.submissions;
 
 export default function SubmissionHistoryRow({
   isMM,
+  isRDM,
   submission,
   finalScore,
   provisionalScore,
@@ -81,7 +82,7 @@ export default function SubmissionHistoryRow({
           </div>
         </div>
         {
-          isLoggedIn && isMM
+          isLoggedIn && (isMM || isRDM)
           && (numWinners > 0 || challengeStatus === CHALLENGE_STATUS.COMPLETED) && (
             <div styleName="col-2 col center">
               <div styleName="mobile-header">Action</div>
@@ -121,6 +122,7 @@ SubmissionHistoryRow.defaultProps = {
 
 SubmissionHistoryRow.propTypes = {
   isMM: PT.bool.isRequired,
+  isRDM: PT.bool.isRequired,
   submission: PT.number.isRequired,
   finalScore: PT.oneOfType([
     PT.number,

diff --git a/src/shared/components/challenge-detail/Submissions/SubmissionRow/index.jsx b/src/shared/components/challenge-detail/Submissions/SubmissionRow/index.jsx
@@ -19,7 +19,7 @@ import SubmissionHistoryRow from './SubmissionHistoryRow';
 import style from './style.scss';
 
 export default function SubmissionRow({
-  isMM, openHistory, member, submissions, score, toggleHistory, challengeStatus,
+  isMM, isRDM, openHistory, member, submissions, score, toggleHistory, challengeStatus,
   isReviewPhaseComplete, finalRank, provisionalRank, onShowPopup, rating, viewAsTable,
   numWinners, auth, isLoggedIn,
 }) {
@@ -166,7 +166,8 @@ export default function SubmissionRow({
                   Time
                 </div>
                 {
-                  isMM && (numWinners > 0 || challengeStatus === CHALLENGE_STATUS.COMPLETED) && (
+                  (isMM || isRDM)
+                  && (numWinners > 0 || challengeStatus === CHALLENGE_STATUS.COMPLETED) && (
                     <div styleName="col-2 col center">
                       Action
                     </div>
@@ -185,6 +186,7 @@ export default function SubmissionRow({
                   <SubmissionHistoryRow
                     isReviewPhaseComplete={isReviewPhaseComplete}
                     isMM={isMM}
+                    isRDM={isRDM}
                     challengeStatus={challengeStatus}
                     submission={submissions.length - index}
                     {...submissionHistory}
@@ -224,6 +226,7 @@ SubmissionRow.defaultProps = {
 
 SubmissionRow.propTypes = {
   isMM: PT.bool.isRequired,
+  isRDM: PT.bool.isRequired,
   openHistory: PT.bool.isRequired,
   member: PT.string.isRequired,
   challengeStatus: PT.string.isRequired,

diff --git a/src/shared/components/challenge-detail/Submissions/index.jsx b/src/shared/components/challenge-detail/Submissions/index.jsx
@@ -6,7 +6,7 @@
 import React from 'react';
 import PT from 'prop-types';
 import moment from 'moment';
-import { isMM as checkIsMM } from 'utils/challenge';
+import { isMM as checkIsMM, isRDM as checkIsRDM } from 'utils/challenge';
 import _ from 'lodash';
 import { connect } from 'react-redux';
 import { config } from 'topcoder-react-utils';
@@ -309,6 +309,7 @@ class SubmissionsComponent extends React.Component {
     } = challenge;
 
     const isMM = this.isMM();
+    const isRDM = checkIsRDM(challenge);
     const isLoggedIn = !_.isEmpty(auth.tokenV3);
     const isReviewPhaseComplete = this.checkIsReviewPhaseComplete();
 
@@ -604,7 +605,7 @@ class SubmissionsComponent extends React.Component {
           }
           {
             ((numWinners > 0 || challenge.status === CHALLENGE_STATUS.COMPLETED)
-            && isMM && isLoggedIn) && (
+            && (isMM || isRDM) && isLoggedIn) && (
               <div styleName="block-download-all">
                 <button
                   disabled={downloadingAll}
@@ -845,6 +846,7 @@ class SubmissionsComponent extends React.Component {
                   submissions={sortedSubmissions}
                   isReviewPhaseComplete={isReviewPhaseComplete}
                   isMM={isMM}
+                  isRDM={isRDM}
                   key={submission.member}
                   {...submission}
                   challengeStatus={challenge.status}

diff --git a/src/shared/components/challenge-detail/Winners/Winner/index.jsx b/src/shared/components/challenge-detail/Winners/Winner/index.jsx
@@ -19,6 +19,7 @@ function getId(submissions, placement) {
 export default function Winner({
   isDesign,
   isMM,
+  isRDM,
   prizes,
   submissions,
   viewable,
@@ -89,7 +90,7 @@ export default function Winner({
       </div>
       <div styleName="download-container">
         {
-        ((!winner.submissionDownloadLink || !viewable) && isMM && isLoggedIn) && (
+        ((!winner.submissionDownloadLink || !viewable) && (isMM || isRDM) && isLoggedIn) && (
           <button
             onClick={() => {
               // download submission
@@ -145,6 +146,7 @@ Winner.defaultProps = {
 Winner.propTypes = {
   isDesign: PT.bool.isRequired,
   isMM: PT.bool.isRequired,
+  isRDM: PT.bool.isRequired,
   prizes: PT.arrayOf(PT.shape()),
   submissions: PT.arrayOf(PT.object).isRequired,
   viewable: PT.bool.isRequired,

diff --git a/src/shared/components/challenge-detail/Winners/index.jsx b/src/shared/components/challenge-detail/Winners/index.jsx
@@ -22,6 +22,7 @@ export default function Winners({
   viewable,
   isDesign,
   isMM,
+  isRDM,
   isLoggedIn,
   auth,
   challengeStatus,
@@ -31,7 +32,7 @@ export default function Winners({
     <div styleName="container">
       {
         ((winners.length > 0 || challengeStatus === CHALLENGE_STATUS.COMPLETED)
-        && isMM && isLoggedIn) && (
+        && (isMM || isRDM) && isLoggedIn) && (
           <div styleName="block-download-all">
             <button
               disabled={downloadingAll}
@@ -80,6 +81,7 @@ export default function Winners({
           <Winner
             isDesign={isDesign}
             isMM={isMM}
+            isRDM={isRDM}
             key={`${w.handle}-${w.placement}`}
             prizes={prizes}
             submissions={submissions}
@@ -101,6 +103,7 @@ Winners.defaultProps = {
   viewable: false,
   isDesign: false,
   isMM: false,
+  isRDM: false,
   isLoggedIn: false,
   challengeStatus: '',
 };
@@ -112,6 +115,7 @@ Winners.propTypes = {
   viewable: PT.bool,
   isDesign: PT.bool,
   isMM: PT.bool,
+  isRDM: PT.bool,
   isLoggedIn: PT.bool,
   challengeStatus: PT.string,
   auth: PT.shape().isRequired,

diff --git a/src/shared/components/challenge-listing/ChallengeCard/Prize/index.jsx b/src/shared/components/challenge-listing/ChallengeCard/Prize/index.jsx
@@ -6,7 +6,7 @@
 
 import PT from 'prop-types';
 import React from 'react';
-import Tooltip from '../../../Tooltip';
+import Tooltip from 'components/Tooltip';
 import Tip from './Tip';
 import './style.scss';
 
@@ -35,6 +35,11 @@ export default function Prize({
     </div>
   );
 
+  function placeArrow(TooltipNode) {
+    const arrow = TooltipNode.querySelector('.rc-tooltip-arrow');
+    arrow.style.left = '33%';
+  }
+
   const component = (
     <div
       // eslint-disable-next-line jsx-a11y/no-noninteractive-tabindex
@@ -43,13 +48,7 @@ export default function Prize({
     >
       {((onlyShowTooltipForPrize && !withoutTooltip)
         ? (
-          <Tooltip
-            content={tip}
-            placeArrow={(TooltipNode) => {
-              const arrow = TooltipNode.querySelector('.rc-tooltip-arrow');
-              arrow.style.left = '33%';
-            }}
-          >
+          <Tooltip content={tip} placeArrow={placeArrow}>
             {prizeUI}
           </Tooltip>
         )

diff --git a/src/shared/components/tc-communities/AccessDenied/index.jsx b/src/shared/components/tc-communities/AccessDenied/index.jsx
@@ -24,7 +24,6 @@ export default function AccessDenied(props) {
     spaceName,
     environment,
     viewportId,
-    children,
   } = props;
 
   if (viewportId) {
@@ -68,7 +67,7 @@ export default function AccessDenied(props) {
           <div styleName="msg">
             You are not authorized to access this page.
           </div>
-          {children}
+          {props.children}
         </div>
       );
     case CAUSE.HAVE_NOT_SUBMITTED_TO_THE_CHALLENGE:

diff --git a/src/shared/components/tc-communities/communities/wipro/Home/index.jsx b/src/shared/components/tc-communities/communities/wipro/Home/index.jsx
@@ -123,7 +123,6 @@ const settings = {
 export default function Home(props) {
   const {
     userId,
-    resetChallengeListing,
   } = props;
 
   return (
@@ -200,7 +199,7 @@ export default function Home(props) {
       >
         <div styleName="ImageTextStyles.linkWrap style.linkWrap">
           <Link
-            onClick={() => resetChallengeListing()}
+            onClick={() => props.resetChallengeListing()}
             styleName="ImageTextStyles.link"
             to="challenges?communityId="
           >

diff --git a/src/shared/containers/ErrorIcons/index.jsx b/src/shared/containers/ErrorIcons/index.jsx
@@ -25,8 +25,8 @@ function Tip({ errors, clear }) {
         &#10799;
       </a>
       {
-        errors.map(({ title, message }) => (
-          <div key={`${title}-${message}`} styleName="item">
+        errors.map(({ title, message }, i) => (
+          <div key={i.toString()} styleName="item">
             <div styleName="title" title={title}>
               {title}
             </div>