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

Initialize Mercurial Gauntlet on story skip #893

Merged
merged 2 commits into from
Jun 22, 2024
Merged

Conversation

SapiensAnatis
Copy link
Owner

Seeing errors in the logs about people attempting to access Mercurial Gauntlet without the required DbQuestWalls initialized. It seems the gap here is story skip - you skip past the MG story but without having the data initialized. Attempting to read the story after this will not send a request to the server.

Will need to run a DB query after this to fix affected saves.

Plus a usual bunch of misc refactoring and global query filter usage.

Seeing errors in the logs about people attempting to access Mercurial
Gauntlet without the required DbQuestWalls initialized. It seems the
gap here is story skip - you skip past the MG story but without having
the data initialized. Attempting to read the story after this will not
send a request to the server.

Will need to run a DB query after this to fix affected saves.

Plus a usual bunch of misc refactoring and global query filter usage.
This gives a warning about using Take without an OrderBy, but the guard
here against additional wall rows is unlikely to be worth it.
@SapiensAnatis SapiensAnatis merged commit 915748b into main Jun 22, 2024
6 checks passed
@SapiensAnatis SapiensAnatis deleted the fix/mg-story-skip branch June 22, 2024 22:33
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.

1 participant