Additional download submission UAT tweaks

Author: jmgasper

__tests__/shared/components/challenge-detail/Submissions/SubmissionRow/SubmissionHistoryRow/index.jsx

@@ -18,6 +18,7 @@ const mockData = {
     tokenV3: 'tokenV3',
   },
   submissionId: '1',
+  isLoggedIn: true,
 };
 
 describe('Matches shallow shapshot', () => {

src/shared/components/challenge-detail/Submissions/SubmissionRow/SubmissionHistoryRow/index.jsx

@@ -31,6 +31,7 @@ export default function SubmissionHistoryRow({
   auth,
   numWinners,
   submissionId,
+  isLoggedIn,
 }) {
   const getInitialReviewResult = () => {
     if (provisionalScore && provisionalScore < 0) return <FailedSubmissionTooltip />;
@@ -80,7 +81,8 @@ export default function SubmissionHistoryRow({
           </div>
         </div>
         {
-          isMM && (numWinners > 0 || challengeStatus === CHALLENGE_STATUS.COMPLETED) && (
+          isLoggedIn && isMM
+          && (numWinners > 0 || challengeStatus === CHALLENGE_STATUS.COMPLETED) && (
             <div styleName="col-2 col center">
               <div styleName="mobile-header">Action</div>
               <button
@@ -114,6 +116,7 @@ SubmissionHistoryRow.defaultProps = {
   finalScore: null,
   provisionalScore: null,
   isReviewPhaseComplete: false,
+  isLoggedIn: false,
 };
 
 SubmissionHistoryRow.propTypes = {
@@ -134,4 +137,5 @@ SubmissionHistoryRow.propTypes = {
   auth: PT.shape().isRequired,
   numWinners: PT.number.isRequired,
   submissionId: PT.string.isRequired,
+  isLoggedIn: PT.bool,
 };

src/shared/components/challenge-detail/Submissions/SubmissionRow/index.jsx

@@ -21,7 +21,7 @@ import style from './style.scss';
 export default function SubmissionRow({
   isMM, openHistory, member, submissions, score, toggleHistory, challengeStatus,
   isReviewPhaseComplete, finalRank, provisionalRank, onShowPopup, rating, viewAsTable,
-  numWinners, auth,
+  numWinners, auth, isLoggedIn,
 }) {
   const {
     submissionTime, provisionalScore, status, submissionId,
@@ -173,7 +173,7 @@ export default function SubmissionRow({
                   )
                 }
                 {
-                  isMM && (
+                  isMM && isLoggedIn && (
                     <div styleName="col">&nbsp;</div>
                   )
                 }
@@ -193,6 +193,7 @@ export default function SubmissionRow({
                     member={member}
                     numWinners={numWinners}
                     auth={auth}
+                    isLoggedIn={isLoggedIn}
                     submissionId={submissionHistory.submissionId}
                   />
                 ))
@@ -218,6 +219,7 @@ SubmissionRow.defaultProps = {
   finalRank: null,
   provisionalRank: null,
   rating: null,
+  isLoggedIn: false,
 };
 
 SubmissionRow.propTypes = {
@@ -259,6 +261,7 @@ SubmissionRow.propTypes = {
   provisionalRank: PT.number,
   onShowPopup: PT.func.isRequired,
   viewAsTable: PT.bool.isRequired,
+  isLoggedIn: PT.bool,
   numWinners: PT.number.isRequired,
   auth: PT.shape().isRequired,
 };

src/shared/components/challenge-detail/Submissions/index.jsx

@@ -309,6 +309,7 @@ class SubmissionsComponent extends React.Component {
     } = challenge;
 
     const isMM = this.isMM();
+    const isLoggedIn = !_.isEmpty(auth.tokenV3);
     const isReviewPhaseComplete = this.checkIsReviewPhaseComplete();
 
     const { field, sort } = this.getSubmissionsSortParam(isMM, isReviewPhaseComplete);
@@ -603,7 +604,7 @@ class SubmissionsComponent extends React.Component {
           }
           {
             ((numWinners > 0 || challenge.status === CHALLENGE_STATUS.COMPLETED)
-            && isMM) && (
+            && isMM && isLoggedIn) && (
               <div styleName="block-download-all">
                 <button
                   disabled={downloadingAll}
@@ -858,6 +859,7 @@ class SubmissionsComponent extends React.Component {
                   viewAsTable={viewAsTable}
                   numWinners={numWinners}
                   auth={auth}
+                  isLoggedIn={isLoggedIn}
                 />
               ))
             )

src/shared/components/challenge-detail/Submissions/style.scss

@@ -477,18 +477,16 @@
     font-size: 15px;
     line-height: 16px;
     color: #137d60;
+    background-color: transparent;
 
     &:hover {
       color: #0ab88a;
     }
 
     &:disabled {
-      color: #767676;
       pointer-events: none;
-
-      &:hover {
-        color: #767676;
-      }
+      background-color: transparent;
+      color: #137d60;
     }
 
     &.MM {

src/shared/components/challenge-detail/Winners/Winner/style.scss

@@ -233,7 +233,7 @@
 .download-container {
   display: flex;
   align-items: center;
-  margin-left: 20px;
+  margin-left: 40px;
 
   .download {
     text-decoration: none;

src/shared/components/challenge-detail/Winners/style.scss

@@ -34,18 +34,16 @@
   font-size: 15px;
   line-height: 16px;
   color: #137d60;
+  background-color: transparent;
 
   &:hover {
     color: #0ab88a;
   }
 
   &:disabled {
-    color: #767676;
     pointer-events: none;
-
-    &:hover {
-      color: #767676;
-    }
+    background-color: transparent;
+    color: #137d60;
   }
 
   &.MM {