DOMjudge / DOMjura

DOMjudge Results Application
https://domjudge.github.com/DOMjura
MIT License
24 stars 1 forks source link

Does not seem to work on Domjudge 5 #3

Closed techhazard closed 8 years ago

techhazard commented 8 years ago

DOMjura does not seem to work on Domjudge v5. I am testing this on the domjudge of the BAPC (which is still running). The only result I get is this: (logged in as admin) snapshot3 While there are 30+ teams in the contest, and team "Team zonder naam" already has more than one correct submission.
The start and end times of the contest are wrong as well: snapshot2

It might just be because the start- and end times are incorrect. domjudge is running in CEST and the PC running domjura is as well.

Does anyone have an idea if it is just the time, or if it is a DJv5 issue?

nickygerritsen commented 8 years ago

It had something to do with the contest api but I'm not sure currently. If you can find Mart Pluijmaekers (he is in Leiden I think): he had the same problem during the EAPC I think

nickygerritsen commented 8 years ago

Ah yes I know now! The API used to return strings even if it were ints but not anymore. This can be fixed by removing toString() in some places

techhazard commented 8 years ago

I'll see if I can fix that, thanks.

techhazard commented 8 years ago

Don't you mean add a toString() in API get requests?

nickygerritsen commented 8 years ago

No in some places I do toString().toInt() and the toString needs to be removed

techhazard commented 8 years ago

I fixed it!

Live still doesn't work though...

nickygerritsen commented 8 years ago

No, live is todo :( thanks for the fix!