Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: Replay Failed Puzzles from Dashboard #1446

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

CloudyDino
Copy link
Contributor

@CloudyDino CloudyDino commented Feb 15, 2025

This PR adds the ability to replay your failed puzzles from the dashboard.

I don't see an API endpoint to get all the failed puzzles from a certain time frame so the only thing I could think of was to go through the history from newest to oldest (since that's how it's paged) and to keep letting the user solve puzzles until they do all in that time frame. I think having an endpoint would make this easier to do in the same way as the web version.

Another issue I haven't figured out how to fix is the dashboard doesn't update after you solve some of your failed problems. Part of it is because we aren't sending a solve request to the API after every solved puzzle and part of it seems to be because we aren't refreshing the page after we return to the dashboard.

Overall, I would love some guidance on this since this is one of the best puzzle features on web.

add.replay.button.with.logic.mp4

@CloudyDino CloudyDino force-pushed the puzzle-replay branch 5 times, most recently from 8ab1a95 to 942479a Compare February 17, 2025 21:45
@veloce
Copy link
Contributor

veloce commented Feb 18, 2025

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 CloudyDino changed the title Replay Failed Puzzles from Dashboard WIP: Replay Failed Puzzles from Dashboard Feb 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants