Replay Failed Puzzles from Dashboard#1446
Conversation
CloudyDino
force-pushed
the
puzzle-replay
branch
5 times, most recently
from
8ab1a95 to
942479a
Compare
|
Thanks for tackling this. I'll review first the hint and rating PRs. In the meantime I think you can get in touch with the server dev team on discord, and ask about an endpoint to get failed puzzles. |
CloudyDino
force-pushed
the
puzzle-replay
branch
2 times, most recently
from
82c10bd to
6b73a43
Compare
CloudyDino
changed the title
|
Hey @CloudyDino, I just noticed that the Puzzle Replay API is now available: https://lichess.org/api#tag/Puzzles/operation/apiPuzzleReplay |
CloudyDino
force-pushed
the
puzzle-replay
branch
2 times, most recently
from
dbbb37d to
c4faeee
Compare
CloudyDino
changed the title
|
@veloce Could you take a look? |
CloudyDino
force-pushed
the
puzzle-replay
branch
from
c4faeee to
9273cbe
Compare
veloce
left a comment
veloce
left a comment
There was a problem hiding this comment.
@CloudyDino thanks for this.
Code looks legitimate. Before accepting this PR, it should have tests in puzzle_screen_test.dart.
Puzzles are important in the app and well tested. Ideally this new feature should be fully covered.
|
Another piece of feedback: the replay failed button is at the bottom of the screen, not very visible. We could maybe have it in the bottom of the dashboard card (where the graphics is)? Then it should not be styled as a FilledButton. |
Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
|
Thanks! Please see the new tests and the new replay button: 
|
|
Hey @CloudyDino was about to add that the replay doesn't update after you solve a previously failed problem but then I saw you had that already as an outstanding issue Just wanted to confirm that was still existing even once the pr was merged and did you or Veloce have any potential ideas how to address it? Worst case scenario could we not just add pull to refresh on the page But I think Veloce already has some automatic refresh logic without the need to pull to refresh on the home page and was curious if that could be borrowed for here |
|
@ijm8710 Yeah the issue still exists. I'll try to take a look if I can work on the pull to refresh or the automated refresh at some point, but I'm getting a bit busy lately. Let me know if I should create another ticket/issue or if you do so! |
4 participants
Close: #2623
This PR adds the ability to replay your failed puzzles from the dashboard.
Using the new Puzzle Replay API: https://lichess.org/api#tag/Puzzles/operation/apiPuzzleReplay
Another issue I haven't figured out how to fix is the dashboard doesn't update after you solve the failed problems.
I'm really hoping we can get this feature on the new mobile app since I love using it on web!
Old recording:
Android.puzzle.replay.webm