Merge pull request #620 from appirio-tech/feature/sup-2881-show-placement-for-content-submissions-only

SUP-2881, Show placements on challenge cards only for "Contest Submis…

Author: vikasrohit

app/directives/challenge-user-place/challenge-user-place.directive.js

@@ -59,7 +59,9 @@
             $scope.imageURL = $scope.challenge.userDetails.submissions[0].submissionImage && $scope.challenge.userDetails.submissions[0].submissionImage.full;
             $scope.selectedImage = $scope.imageURL;
 
-            $scope.challenge.highestPlacement = _.max($scope.challenge.userDetails.submissions, 'placement').placement;
+            $scope.challenge.highestPlacement = _.min($scope.challenge.userDetails.submissions.filter(function(submission) {
+              return submission.type === CONSTANTS.SUBMISSION_TYPE_CONTEST && submission.placement;
+            }), 'placement').placement;
 
             if ($scope.challenge.highestPlacement == 1) {
               $scope.challenge.wonFirst = true;

app/services/challenge.service.js

@@ -190,7 +190,7 @@
             challenge.thumbnailId = challenge.userDetails.submissions[0].id;
 
             challenge.highestPlacement = _.min(challenge.userDetails.submissions.filter(function(submission) {
-              return submission.placement;
+              return submission.type === CONSTANTS.SUBMISSION_TYPE_CONTEST && submission.placement;
             }), 'placement').placement;
 
             if (challenge.highestPlacement == 1) {

app/topcoder.constants.js

@@ -32,7 +32,8 @@ angular.module("CONSTANTS", [])
 	"BUSY_PROGRESS_MESSAGE": "Processing..",
 	"REGISTRATION": "REGISTRATION",
 	"CODING": "CODING",
-	"REGISTERED": "REGISTERED"
+	"REGISTERED": "REGISTERED",
+	"SUBMISSION_TYPE_CONTEST": "Contest Submission"
 })
 
 ;

assets/css/directives/design-challenge-user-place.scss

@@ -11,7 +11,7 @@ design-challenge-user-place {
     .place {
       &.completed, &.passed, &.didnt {
         margin-bottom: -36px;
-        margin-top: 40px;
+        margin-top: 52px;
       }
       margin-bottom: 8px;
       margin-top: 8px;

config.js

@@ -42,7 +42,10 @@ module.exports = function() {
         CODING: 'CODING',
 
         // users' status
-        REGISTERED: 'REGISTERED'
+        REGISTERED: 'REGISTERED',
+
+        // submission type
+        SUBMISSION_TYPE_CONTEST: 'Contest Submission'
 
       }
     },
@@ -88,7 +91,10 @@ module.exports = function() {
         CODING: 'CODING',
 
         // users' status
-        REGISTERED: 'REGISTERED'
+        REGISTERED: 'REGISTERED',
+
+        // submission type
+        SUBMISSION_TYPE_CONTEST: 'Contest Submission'
 
       }
     },
@@ -134,7 +140,10 @@ module.exports = function() {
         CODING: 'CODING',
 
         // users' status
-        REGISTERED: 'REGISTERED'
+        REGISTERED: 'REGISTERED',
+
+        // submission type
+        SUBMISSION_TYPE_CONTEST: 'Contest Submission'
 
       }
     },
@@ -180,7 +189,10 @@ module.exports = function() {
         CODING: 'CODING',
 
         // users' status
-        REGISTERED: 'REGISTERED'
+        REGISTERED: 'REGISTERED',
+
+        // submission type
+        SUBMISSION_TYPE_CONTEST: 'Contest Submission'
 
       }
     },
@@ -226,7 +238,10 @@ module.exports = function() {
         CODING: 'CODING',
 
         // users' status
-        REGISTERED: 'REGISTERED'
+        REGISTERED: 'REGISTERED',
+
+        // submission type
+        SUBMISSION_TYPE_CONTEST: 'Contest Submission'
 
       }
     }