-
Notifications
You must be signed in to change notification settings - Fork 34
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
[Fix] Clear cache and storage data on new browser game window #1194
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested logging into DFK, closing the game, and re-launching. Confirmed I was still logged in on re-launch
@@ -145,14 +146,16 @@ const openNewBrowserGameWindow = async ( | |||
const hpOverlay = await getHpOverlay() | |||
|
|||
return new Promise((res) => { | |||
const partition = `persist:${appName}` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense. Code looks good! 🚀
This should clear the cache of the browser windows when launching games,
It is a workaround for some games like Champz that had issues after games updates.
Another idea would be to clear the cache only when cleaning the hyperplay cache but that might not be intuitive for some users.
How to Test
Not sure since you would need to have an old version of a game loaded, then on the server the game gets updated and then the game launched through hyperplay to trigger the cache clean and be sure that the game would be launched on the latest version.
Use the following Checklist if you have changed something on the Backend or Frontend: