Fix for download button not showing up on winners tab

Author: jmgasper

src/shared/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,

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,

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,

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,

src/shared/containers/challenge-detail/index.jsx

@@ -8,7 +8,7 @@
 
 import _ from 'lodash';
 import communityActions from 'actions/tc-communities';
-import { isMM as checkIsMM } from 'utils/challenge';
+import { isMM as checkIsMM, isRDM as checkIsRDM } from 'utils/challenge';
 import LoadingPagePlaceholder from 'components/LoadingPagePlaceholder';
 import pageActions from 'actions/page';
 import ChallengeHeader from 'components/challenge-detail/Header';
@@ -439,6 +439,7 @@ class ChallengeDetailPageContainer extends React.Component {
 
     const isEmpty = _.isEmpty(challenge);
     const isMM = checkIsMM(challenge);
+    const isRDM = checkIsRDM(challenge);
     const isLegacyMM = isMM && Boolean(challenge.roundId);
 
     if (isLoadingChallenge || isLoadingTerms) {
@@ -661,6 +662,7 @@ class ChallengeDetailPageContainer extends React.Component {
                 submissions={challenge.submissions}
                 isDesign={track.toLowerCase() === 'design'}
                 isMM={isMM}
+                isRDM={isRDM}
                 isLoggedIn={isLoggedIn}
                 auth={auth}
               />