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

Offline puzzle buffer should be ~200, not 50 #2256

Open
aronhegedus opened this issue Dec 5, 2022 · 0 comments · May be fixed by #2315
Open

Offline puzzle buffer should be ~200, not 50 #2256

aronhegedus opened this issue Dec 5, 2022 · 0 comments · May be fixed by #2315

Comments

@aronhegedus
Copy link

Is your feature request related to a problem? Please describe.
I'm frustrated when I'm on a plane and I only have 50 offline puzzles to solve. I get through them, and then want to do more.

I also get into situations where the offline puzzles don't refresh for some reason, even when I was online, and that leads to me not being able to solve puzzles when I'm on a tube.

Describe the solution you'd like
Increase the number of offline puzzles to 200, form the current 50.

Describe alternatives you've considered
Have considered having a mode where you can replay the puzzles that you've gotten wrong in offline mode, which would increase the amount of time that you could spend offline and play puzzles. I've rejected that idea because that would be much more difficult to implement.

Have considered adding a setting feature where a user can toggle how many puzzles they'd like to save offline.

Additional context
This should just boil down to changing puzzleBufferLen: 50, to a higher number.

One drawback could be a small slowdown while getting the offline puzzles. I don't know exactly how high the payload is for 1 puzzle from a server, but I feel like it shouldn't be that much.

@Rica320 Rica320 linked a pull request Feb 18, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants