Protichůdné hlášky při zablokovaném odevzdávání

[BorysekOndrej]

https://imgur.com/a/9zyqoJt

[jirkadanek]

Zkusim se na to podivat. Zvladl jsem to zreprodukovat s lokalnim frontendem a backendem z mastru.

[jiridanek]

Ahoj, koukal jsem na to, a myslim si, ze vim v cem je problem.

Web-frontend ocekava {result: {'next': }} ve vysledku, ale web-backend tohle pole nikdy neposila, takze frontend vzdy vypise ze next odevzdani je mozne ihned. To je kvuli tomu ze neni uplne doimplementovane https://github.com/fi-ksi/web-frontend/issues/301.

Dale, protoze prislo {'result': {'type': 'error'}}, web-frontend nenastavi script_message_output, a proto se na webu vypise to "{null}".

Fixnul bych to tak, ze

1. ve web-frontend bych upravil *-module.js compoenty tak, aby v handlovani response byl identicky kod, koukal jsem na diffy a myslim ze pro rozdily tam neni zadny duvod. S vyjimkou code-module.js, ten bych nechal byt, protoze ten se od tech ostatnich lisi vice a deji se tam veci navic.
2. potom bych opravil vypisovani toho next, aby hlaska byla gramaticky spravne. Momentalne se vypise cas v prvnim pade, coz nesedi, neco jako "Dalsi odevzdani mozne 23. unor 2020 14.00".
3. nakonec ten "null". Podivam se jestli nemuze defaultni hodnota pro script_message_output byt "", jestli se to chova v js a v template taky jako false. Pokud jo, udelal bych to tak, pokud ne, vymyslim neco jineho.

Do web-backend bych pridal aby se posilal 'next' pokud se jedna o 20 odevzdani nebo o zapovezene odevzdani z duvodu prekroceni kvoty.

[BorysekOndrej]

Ahoj Jirko, je super žes na to mrkl. Moc se omlouvám, že odpovídám až teď, mám toho hodně. Jakékoliv bugfixy rádi uvítáme a po rychlé kontrole zamergujeme. Úpravy frontendu do masteru ti můžu potvrdit a nasadit já, úpravy backendu Honza H. Na nasazení testovací verze frontendu i backendu máme testovací server, pokud bys chtěl přístup napiš mi prosím mail a zařídím ti ho. Díky.

Co se konkrétního navrhnutého řešení týče, zní dobře, jsem pro.