Merge pull request #6974 from topcoder-platform/develop

PROD - hide download buttons on submissions for Topcrowd / Phoenix challenges
topcoder-platform · Apr 8, 2024 · 79c1eb2 · 79c1eb2
2 parents 94b5fa2 + f77c875
commit 79c1eb2
Show file tree

Hide file tree

Showing 7 changed files with 84 additions and 41 deletions.
diff --git a/src/shared/components/SubmissionManagement/Submission/index.jsx b/src/shared/components/SubmissionManagement/Submission/index.jsx
@@ -27,6 +27,7 @@ import './styles.scss';
 
 export default function Submission(props) {
   const {
+    challenge,
     submissionObject,
     showScreeningDetails,
     track,
@@ -40,6 +41,15 @@ export default function Submission(props) {
   const onDownloadSubmission = onDownload.bind(1, submissionObject.id);
   const safeForDownloadCheck = safeForDownload(submissionObject.url);
 
+  // Determine if a challenge is for Topcrowd so we can edit the UI accordingly
+  let isTopCrowdChallenge = false;
+  if (challenge) {
+    const isTopCrowdChallengeData = _.find(challenge.metadata, { name: 'is_platform' });
+    if (isTopCrowdChallengeData) {
+      isTopCrowdChallenge = isTopCrowdChallengeData.value;
+    }
+  }
+
   return (
     <tr styleName="submission-row">
       <td styleName="id-col">
@@ -72,12 +82,16 @@ export default function Submission(props) {
        }
       <td styleName="action-col">
         <div>
-          <button
-            onClick={() => onDownloadSubmission(submissionObject.id)}
-            type="button"
-          >
-            { safeForDownloadCheck === true && <DownloadIcon /> }
-          </button>
+          { !isTopCrowdChallenge
+            ? (
+              <button
+                onClick={() => onDownloadSubmission(submissionObject.id)}
+                type="button"
+              >
+                { safeForDownloadCheck === true && <DownloadIcon /> }
+              </button>
+            )
+            : <span /> }
           { /*
              TODO: At the moment we just fetch downloads from the legacy
                Topcoder Studio API, and we don't need any JS code to this.
@@ -121,6 +135,7 @@ Submission.defaultProps = {
 };
 
 Submission.propTypes = {
+  challenge: PT.shape().isRequired,
   submissionObject: PT.shape({
     id: PT.string,
     legacySubmissionId: PT.string,

diff --git a/src/shared/components/SubmissionManagement/SubmissionManagement/index.jsx b/src/shared/components/SubmissionManagement/SubmissionManagement/index.jsx
@@ -171,6 +171,7 @@ export default function SubmissionManagement(props) {
         {!loadingSubmissions
            && (
            <SubmissionsTable
+             challenge={challenge}
              submissionObjects={submissions}
              showDetails={showDetails}
              track={track}
@@ -212,13 +213,13 @@ SubmissionManagement.defaultProps = {
 };
 
 SubmissionManagement.propTypes = {
+  challenge: PT.shape().isRequired,
   showDetails: PT.shape().isRequired,
   onDelete: PT.func,
   onlineReviewUrl: PT.string,
   helpPageUrl: PT.string,
   onDownload: PT.func,
   onShowDetails: PT.func,
-  challenge: PT.shape().isRequired,
   submissions: PT.arrayOf(PT.shape()),
   loadingSubmissions: PT.bool,
   challengeUrl: PT.string,

diff --git a/src/shared/components/SubmissionManagement/SubmissionsTable/index.jsx b/src/shared/components/SubmissionManagement/SubmissionsTable/index.jsx
@@ -26,6 +26,7 @@ import './styles.scss';
 
 export default function SubmissionsTable(props) {
   const {
+    challenge,
     submissionObjects,
     showDetails,
     track,
@@ -56,6 +57,7 @@ export default function SubmissionsTable(props) {
 
       const submission = (
         <Submission
+          challenge={challenge}
           submissionObject={subObject}
           showScreeningDetails={showDetails[subObject.id]}
           track={track}
@@ -140,6 +142,7 @@ SubmissionsTable.defaultProps = {
 };
 
 SubmissionsTable.propTypes = {
+  challenge: PT.shape().isRequired,
   submissionObjects: PT.arrayOf(SubShape),
   showDetails: PT.shape().isRequired,
   track: PT.string.isRequired,

diff --git a/src/shared/components/challenge-detail/MySubmissions/SubmissionsList/index.jsx b/src/shared/components/challenge-detail/MySubmissions/SubmissionsList/index.jsx
@@ -187,6 +187,15 @@ class SubmissionsListView extends React.Component {
       timeClicked: false,
     };
 
+    // Determine if a challenge is for Topcrowd so we can edit the UI accordingly
+    let isTopCrowdChallenge = false;
+    const isTopCrowdChallengeData = _.find(challenge.metadata, { name: 'is_platform' });
+    if (isTopCrowdChallengeData) {
+      isTopCrowdChallenge = isTopCrowdChallengeData.value;
+    } else {
+      isTopCrowdChallenge = false;
+    }
+
     return (
       <div styleName="wrapper">
         <div styleName="submission-table">
@@ -432,25 +441,29 @@ class SubmissionsListView extends React.Component {
                       <span>{moment(mySubmission.submissionTime).format('MMM DD, YYYY HH:mm:ss')}</span>
                     </div>
                     <div styleName="submission-table-column column-2-4">
-                      <button
-                        onClick={() => {
-                          // download submission
-                          const submissionsService = getService(auth.tokenV3);
-                          submissionsService.downloadSubmission(mySubmission.submissionId)
-                            .then((blob) => {
-                              const url = window.URL.createObjectURL(new Blob([blob]));
-                              const link = document.createElement('a');
-                              link.href = url;
-                              link.setAttribute('download', `submission-${mySubmission.submissionId}.zip`);
-                              document.body.appendChild(link);
-                              link.click();
-                              link.parentNode.removeChild(link);
-                            });
-                        }}
-                        type="button"
-                      >
-                        <DownloadIcon />
-                      </button>
+                      { !isTopCrowdChallenge
+                        ? (
+                          <button
+                            onClick={() => {
+                              // download submission
+                              const submissionsService = getService(auth.tokenV3);
+                              submissionsService.downloadSubmission(mySubmission.submissionId)
+                                .then((blob) => {
+                                  const url = window.URL.createObjectURL(new Blob([blob]));
+                                  const link = document.createElement('a');
+                                  link.href = url;
+                                  link.setAttribute('download', `submission-${mySubmission.submissionId}.zip`);
+                                  document.body.appendChild(link);
+                                  link.click();
+                                  link.parentNode.removeChild(link);
+                                });
+                            }}
+                            type="button"
+                          >
+                            <DownloadIcon />
+                          </button>
+                        )
+                        : <span /> }
 
                       <button onClick={() => selectSubmission(mySubmission)} type="button">
                         <ZoomIcon styleName="icon-zoom" />
@@ -523,13 +536,7 @@ SubmissionsListView.defaultProps = {
 SubmissionsListView.propTypes = {
   selectSubmission: PT.func,
   challengesUrl: PT.string.isRequired,
-  challenge: PT.shape({
-    id: PT.any,
-    checkpoints: PT.arrayOf(PT.object),
-    submissions: PT.arrayOf(PT.object),
-    submissionViewable: PT.string,
-    registrants: PT.any,
-  }).isRequired,
+  challenge: PT.shape().isRequired,
   hasRegistered: PT.bool.isRequired,
   unregistering: PT.bool.isRequired,
   submissionEnded: PT.bool.isRequired,

diff --git a/src/shared/components/challenge-detail/MySubmissions/index.jsx b/src/shared/components/challenge-detail/MySubmissions/index.jsx
@@ -125,13 +125,7 @@ MySubmissionsView.defaultProps = {
 
 MySubmissionsView.propTypes = {
   challengesUrl: PT.string.isRequired,
-  challenge: PT.shape({
-    id: PT.any,
-    checkpoints: PT.arrayOf(PT.object),
-    submissions: PT.arrayOf(PT.object),
-    submissionViewable: PT.string,
-    registrants: PT.any,
-  }).isRequired,
+  challenge: PT.shape().isRequired,
   hasRegistered: PT.bool.isRequired,
   unregistering: PT.bool.isRequired,
   submissionEnded: PT.bool.isRequired,

diff --git a/src/shared/components/challenge-detail/Registrants/index.jsx b/src/shared/components/challenge-detail/Registrants/index.jsx
@@ -58,6 +58,7 @@ export default class Registrants extends React.Component {
 
     this.getCheckPoint = this.getCheckPoint.bind(this);
     this.getCheckPointDate = this.getCheckPointDate.bind(this);
+    this.getSubmissionDate = this.getSubmissionDate.bind(this);
     this.getFlagFirstTry = this.getFlagFirstTry.bind(this);
     this.sortRegistrants = this.sortRegistrants.bind(this);
     this.getRegistrantsSortParam = this.getRegistrantsSortParam.bind(this);
@@ -128,6 +129,23 @@ export default class Registrants extends React.Component {
     return final;
   }
 
+  /**
+    * Get the submission date of a registrant (used when viewing the registrants tab anonymously)
+    * @param {Object} registrant the registrant to return the submission date for
+    */
+  getSubmissionDate(registrant) {
+    const {
+      statisticsData,
+    } = this.props;
+    console.log(JSON.stringify(statisticsData, null, 4));
+    let submissionDate;
+    const statistic = (statisticsData || []).find(x => x.handle === registrant.memberHandle);
+    if (statistic && statistic.submissions && statistic.submissions.length > 0) {
+      submissionDate = statistic.submissions.sort()[0].created;
+    }
+    return submissionDate;
+  }
+
   /**
     * Check if it have flag for first try
     * @param {Object} registrant registrant info
@@ -413,7 +431,10 @@ export default class Registrants extends React.Component {
                if (checkpoint) {
                  checkpoint = formatDate(checkpoint);
                }
-               const final = this.getFinal(r);
+               let final = this.getFinal(r);
+               if (!final) {
+                 final = this.getSubmissionDate(r);
+               }
 
                return (
                  <div styleName="row" key={r.memberHandle} role="row">
@@ -523,6 +544,7 @@ Registrants.propTypes = {
     type: PT.string,
     track: PT.string,
   }).isRequired,
+  statisticsData: PT.arrayOf(PT.shape()).isRequired,
   results: PT.arrayOf(PT.shape()),
   checkpointResults: PT.shape(),
   registrants: PT.arrayOf(PT.shape()),

diff --git a/src/shared/containers/challenge-detail/index.jsx b/src/shared/containers/challenge-detail/index.jsx
@@ -565,6 +565,7 @@ class ChallengeDetailPageContainer extends React.Component {
                   )
                 }
                 results={results2}
+                statisticsData={statisticsData}
                 registrantsSort={registrantsSort}
                 notFoundCountryFlagUrl={notFoundCountryFlagUrl}
                 onGetFlagImageFail={(countryInfo) => {