Skip to content

Additional changes to the Export contest submissions details script#295

Open
jalsol wants to merge 2 commits intomasterfrom
fix-export-submissions-details
Open

Additional changes to the Export contest submissions details script#295
jalsol wants to merge 2 commits intomasterfrom
fix-export-submissions-details

Conversation

@jalsol
Copy link
Contributor

@jalsol jalsol commented Feb 9, 2023

Description

What

  • Create an additional row above the test case rows for each submission
  • Display error messages
  • Add a status column to submissions and test cases
  • Only export submissions of live participations

image

image

Why

  • Creating a row for submission information allows the display of submissions with no test case run (CE, IE, etc) and error messages
  • Submissions other than those from live participations should not be considered when exported for an official contest

How Has This Been Tested?

Tested on my local version of VNOI. Ran on contests hello22 and contest_testing_01.

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

  • I have explained the purpose of this PR.
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the README/documentation
  • Any dependent changes have been merged and published in downstream modules
  • Informed of breaking changes, testing and migrations (if applicable).
  • Attached screenshots (if applicable).

By submitting this pull request, I confirm that my contribution is made under the terms of the AGPL-3.0 License.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant