search

papi-web-org / papi-web

Papi-web
7 stars 3 forks source link

Replace Django with Litestar and add a few features #28

Closed Amaras closed 5 months ago

Amaras commented 6 months ago

This PR replaces Django with Litestar (closes #24), and adds the following changes:

  1. Adds checkmarks in the playerlist (boards screen at the start of the tournament): players with filled checkmarks are checked-in (wonderful when coupled with ChessEvent).
  2. [WIP] Add flags to track illegal moves (on boards screens with update = on and record_illegal_moves = on): click on empty flag to add an illegal move flag to a player, click on a filled flag to remove it (currently not done).

More features may arrive before merging.

adrienave commented 5 months ago

Just noticed the PR has been merge before my review, bad timing from myself. It's okay, everything was about code smells / refactoring, nothing wrong in the logic 👍