search

lichess-org / lichobile

lichess.org mobile application
https://lichess.org/mobile
GNU General Public License v3.0
2.04k stars 318 forks source link

Tournament creation is broken #719

Closed ornicar closed 6 years ago

ornicar commented 6 years ago
ornicar commented 6 years ago

quoting myself from slack:

The position field was completely bugged out, I fixed it and broke BC
before, the position was an ECO code such as "B01".
The problem is that it's ambiguous, there are many openings and positions for each ECO code
so now the position field is a FEN.
https://github.com/ornicar/lila/blob/f973660244fb72932c392c4c63adb04c70b04ff4/app/views/tournament/startingPosition.scala.html#L9
or inspect https://lichess.org/tournament/new
I will have the server ignore invalid FENs (such as B01) so mobile users can at least create tournaments with standard position
veloce commented 6 years ago

Fixed by #726