-
Notifications
You must be signed in to change notification settings - Fork 186
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
Reset Vuex state when returning to game selection to free up memory #1385
Reset Vuex state when returning to game selection to free up memory #1385
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.
I'm not sure what "start resetting" in the PR title refers to, but yes, this looks like a good beginning of what I had in mind. However, if you check the ticket, my original idea was to call a reset method on the main/index store, and let it handle resetting the substores. Is there a reason why this now directly calls reset on profile substore? Are you planning adding the resets of the other stores in this PR or a separate one?
1dafecf
to
fc95df9
Compare
bd9342e
to
5aa8b0e
Compare
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.
The changes shouldn't break remembering the last selected profile.
5aa8b0e
to
03e2f40
Compare
I'll approve this PR when #1410 is approved. |
…onScreen is created
03e2f40
to
762be48
Compare
…in page components
There was still a bug where returning to profile selections screen and then moving back to manager screen with the same profile (without changing the active profile in between) displayed an empty local mod list. Also if the TsModsModule can't be reset here, it somewhat waters down the another half of the purpose of these changes, which was to free up memory when it's not needed. The things that would have gotten reset are very small compared to the large mod lists. Full resetting is still done if the active game is changed.
Add data resetting to vuex storage modules and use the functionality in page components
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.
Though approved, I'll delay merging this for now since we want to keep the "release pipeline" clear. This should be squash-merged when the time comes.
No description provided.