Skip to content

Conversation

vmcj
Copy link
Member

@vmcj vmcj commented Sep 28, 2025

We get this case for jury imports where the same team DOMjudge uploads in multiple contests (dress + finals) and the first "finals" submission would show a diff with the last jury submission of the dress contest.

We could also have this if a team is in 2 contests at the same time but although in theory possible this shouldn't happen in practice.

We get this case for jury imports where the same team DOMjudge
uploads in multiple contests (dress + finals) and the first "finals" submission
would show a diff with the last jury submission of the dress contest.

We could also have this if a team is in 2 contests at the same time but
although in theory possible this shouldn't happen in practice.
@meisterT
Copy link
Member

We get this case for jury imports where the same team DOMjudge uploads in multiple contests (dress + finals) and the first "finals" submission would show a diff with the last jury submission of the dress contest.

Huh, did you test with the same problem in both contests?

@vmcj
Copy link
Member Author

vmcj commented Sep 28, 2025

We get this case for jury imports where the same team DOMjudge uploads in multiple contests (dress + finals) and the first "finals" submission would show a diff with the last jury submission of the dress contest.

Huh, did you test with the same problem in both contests?

In my test db I have the same problem in multiple contests yes. Which makes this even more of an edge case.

@meisterT
Copy link
Member

It's a good change anyway, just wanted to understand what caused this

@vmcj vmcj added this to the 9.0 milestone Sep 28, 2025
@meisterT meisterT added this pull request to the merge queue Sep 29, 2025
Merged via the queue into DOMjudge:main with commit 201b40b Sep 29, 2025
36 checks passed
@vmcj vmcj deleted the source_code_diff branch September 29, 2025 18:13
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.

3 participants