New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix #1995: Feature for Baseline Entries #2076
Conversation
Issue#1995 Frontend badge
Issue#1995 payload bug
Modified Tests for Baseline Submissions
@deshraj @RishabhJain2018 Please review! |
@vkartik97 Let's merge this by tomorrow. Please make the changes. It's a really nice feature. :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@vkartik97 I'm merging this for now since we need it urgently. Please add tests for this before working on any other feature.
@@ -280,6 +280,9 @@ def change_submission_data_and_visibility( | |||
"error": "Sorry, cannot accept submissions since challenge phase is not public" | |||
} | |||
return Response(response_data, status=status.HTTP_403_FORBIDDEN) | |||
elif request.data.get("is_baseline"): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@vkartik97 Can you please add tests for this condition.
for data in sorted_leaderboard_data: | ||
if data["submission__participant_team__team_name"] in team_list: | ||
continue | ||
elif data['submission__is_baseline'] is True: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add tests for this condition.
closes: #1995
Deliverables:
baseline_submission
change_submission_data_and_visibility
to change to toggle the value of the fieldbaseline
, only for hosts.My submissions
table for challenge hosts only, so that they can toggle this functionality.baseline
to show which entries are baseline.Screenshots: