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.

Sign up for GitHub

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

Jump to bottom

[Mobile] incorrect number of puzzles after solving puzzles with the batch endpoint #14773

Closed
veloce opened this issue Feb 29, 2024 · 3 comments
Closed

[Mobile] incorrect number of puzzles after solving puzzles with the batch endpoint #14773

veloce opened this issue Feb 29, 2024 · 3 comments
Labels
bug

Comments

@veloce
Copy link
Contributor

veloce commented Feb 29, 2024

After using the batch endpoint to solve many puzzles at once (played offline on the app), it looks that number of solved puzzles only increase by 1, regardless of the number of solved puzzles.

A detailed report here: lichess-org/mobile#578
@veloce veloce added the bug label Feb 29, 2024
@ornicar ornicar closed this as completed in e6ce724 Mar 1, 2024
ornicar added a commit to lenguyenthanh/lila that referenced this issue Mar 1, 2024
@ornicar
Merge branch 'master' into rewrite-infix
5c795f9 
* master:
  fix puzzle batch solve - closes lichess-org#14773 - closes lichess-org/mobile#578
  use traverse_
  upgrade scalachess
  tweak scala import
  improve fide sync monitoring
  better import kamon functions
  scala 3.3.3
@Robmonster
Copy link

Thanks for the fix! Is there a way of knowing when this change makes it to the live server?

@Carbrex
Copy link
Contributor

Carbrex commented Mar 1, 2024

Thanks for the fix! Is there a way of knowing when this change makes it to the live server?

https://lichess.org/source to know which commit is it currently running.

ornicar added a commit that referenced this issue Mar 2, 2024
@ornicar
Merge branch 'master' into relay-ui-v2
2c439ea 
* master:
  The FID Federation is missing (#14778)
  Min rating FIDE is now 1400
  fix puzzle batch solve - closes #14773 - closes lichess-org/mobile#578
  use traverse_
  upgrade scalachess
  tweak scala import
  improve fide sync monitoring
  better import kamon functions
  scala 3.3.3
  encourage setting fide ids rather than manual tags
  coloured titles and /fide links in broadcast leaderboard
  link broadcast players to /fide/:id
@Robmonster
Copy link

Thanks for the fix! Is there a way of knowing when this change makes it to the live server?

https://lichess.org/source to know which commit is it currently running.

I'm struggling to understand how the commit names work, so I'll just wait a while and then retest.,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Robmonster @veloce @Carbrex