Skip to content
This repository was archived by the owner on Mar 4, 2025. It is now read-only.

Commit 1a43168

Browse files
author
vikasrohit
committed
SUP-3004, [Profile] Still showing placements outside of final submission range
-- Fixed showing placement outside of the winning placement boundaries and shown 'Passed Review' instead of placement for such challenges.
1 parent 6c2cee0 commit 1a43168

File tree

3 files changed

+4
-8
lines changed

3 files changed

+4
-8
lines changed

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

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -59,14 +59,6 @@ import _ from 'lodash'
5959
$scope.challenge.thumbnailId = $scope.challenge.userDetails.submissions[0].id
6060
$scope.imageURL = $scope.challenge.userDetails.submissions[0].submissionImage && $scope.challenge.userDetails.submissions[0].submissionImage.full
6161
$scope.selectedImage = $scope.imageURL
62-
63-
$scope.challenge.highestPlacement = _.min($scope.challenge.userDetails.submissions.filter(function(submission) {
64-
return submission.type === CONSTANTS.SUBMISSION_TYPE_CONTEST && submission.placement
65-
}), 'placement').placement
66-
67-
if ($scope.challenge.highestPlacement == 1) {
68-
$scope.challenge.wonFirst = true
69-
}
7062
}
7163
}
7264

app/directives/challenge-user-place/design-challenge-user-place.jade

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
p.place(ng-class="{ 'first-place': challenge.wonFirst }", ng-show="challenge.highestPlacement") {{challenge.highestPlacement}}#[span {{challenge.highestPlacement | ordinal:true}}] Place
33
p.place.didnt(ng-show="challenge.userStatus === 'NOT_FINISHED'") Didn't Finish
44
p.place.passed(ng-show="challenge.userStatus === 'PASSED_SCREENING'") Passed Screening
5+
p.place(ng-show="!challenge.highestPlacement && challenge.userStatus === 'PASSED_REVIEW'") Passed Review
56
p.place.completed(ng-show="challenge.userStatus === 'COMPLETED'") COMPLETED
67

78
.thumbnail(ng-click="!challenge.isPrivate && imageURL && openLightbox()", ng-class="{hidden: challenge.userStatus !== 'PASSED_REVIEW'}")
@@ -23,6 +24,7 @@
2324
p.place(ng-class="{ 'first-place': challenge.wonFirst }", ng-show="challenge.highestPlacement") {{challenge.highestPlacement}}#[span {{challenge.highestPlacement | ordinal:true}}] Place
2425
p.place.didnt(ng-show="challenge.userStatus === 'NOT_FINISHED'") Didn't Finish
2526
p.place.passed(ng-show="challenge.userStatus === 'PASSED_SCREENING'") Passed Screening
27+
p.place(ng-show="!challenge.highestPlacement && challenge.userStatus === 'PASSED_REVIEW'") Passed Review
2628
p.place.completed(ng-show="challenge.userStatus === 'COMPLETED'") COMPLETED
2729
p.date-completed {{challenge.submissionEndDate | date : 'MMMM yyyy'}}
2830

app/directives/challenge-user-place/dev-challenge-user-place.jade

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
p.place(ng-class="{ 'first-place': challenge.wonFirst }", ng-show="challenge.highestPlacement") {{challenge.highestPlacement}}#[span {{challenge.highestPlacement | ordinal:true}}] Place
33
p.place(ng-show="challenge.userStatus === 'NOT_FINISHED'") Didn't Finish
44
p.place(ng-show="challenge.userStatus === 'PASSED_SCREENING'") Passed Screening
5+
p.place(ng-show="!challenge.highestPlacement && challenge.userStatus === 'PASSED_REVIEW'") Passed Review
56
p.place(ng-show="challenge.userStatus === 'COMPLETED'") COMPLETED
67

78
.challenge-score(ng-hide="challenge.userStatus !== 'PASSED_REVIEW'")
@@ -14,6 +15,7 @@
1415
p.place(ng-class="{ 'first-place': challenge.wonFirst }", ng-show="challenge.highestPlacement") {{challenge.highestPlacement}}#[span {{challenge.highestPlacement | ordinal:true}}] Place
1516
p.place(ng-show="challenge.userStatus === 'NOT_FINISHED'") Didn't Finish
1617
p.place(ng-show="challenge.userStatus === 'PASSED_SCREENING'") Passed Screening
18+
p.place(ng-show="!challenge.highestPlacement && challenge.userStatus === 'PASSED_REVIEW'") Passed Review
1719
p.place(ng-show="challenge.userStatus === 'COMPLETED'") Completed
1820
p.date-completed {{challenge.submissionEndDate | date : 'MMMM yyyy'}}
1921

0 commit comments

Comments
 (0)