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

Auto next day for winner broke #1234

Open
ctm opened this issue Oct 26, 2023 · 1 comment
Open

Auto next day for winner broke #1234

ctm opened this issue Oct 26, 2023 · 1 comment
Assignees
Labels
bug Something isn't working can't reproduce Can't reproduce regression Something that formerly worked

Comments

@ctm
Copy link
Owner

ctm commented Oct 26, 2023

Fix so that if someone wins a tournament and then creates a new one it defaults to the right time.

This used to work as long as the person logged in to the lobby before the game started and stayed in the lobby without refreshing until asking to create a tournament. I don't know when it broke, but IIRC, some people were reporting it having broken when it seemed to work for me, but the last couple of times I've won it's happened to me, too, and jpmassar reported it tonight:

jpmassar: The auto-next-day-same-time-for-next-tournament feature seems permanently broken?

If I can reproduce it, it'll be trivial to fix. If I can't reproduce it, I'll add a bunch of tracing and then it should be trivial to fix once I get the logs.

@ctm ctm added bug Something isn't working high priority Should be done fairly soon easy Trivial to do (even when tired!) and semi-worthwhile regression Something that formerly worked labels Oct 26, 2023
@ctm ctm self-assigned this Oct 26, 2023
@ctm
Copy link
Owner Author

ctm commented Nov 17, 2023

I was able to get it to trigger locally, but not on my first try. I should be able to track this down though now that I'm no longer working on table splitting.

@ctm ctm removed high priority Should be done fairly soon easy Trivial to do (even when tired!) and semi-worthwhile labels Jan 12, 2024
@ctm ctm added the can't reproduce Can't reproduce label Mar 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working can't reproduce Can't reproduce regression Something that formerly worked
Projects
None yet
Development

No branches or pull requests

1 participant