Skip to content

Add feature to save history like bookmarks - #1011

Merged
slprime merged 3 commits into
GTNewHorizons:masterfrom
vcwdfca:Save-the-history
Aug 26, 2026
Merged

Add feature to save history like bookmarks#1011
slprime merged 3 commits into
GTNewHorizons:masterfrom
vcwdfca:Save-the-history

Conversation

@vcwdfca

@vcwdfca vcwdfca commented Aug 23, 2026

Copy link
Copy Markdown

Summary

closes #1010

What

This PR allows history to be saved like bookmarks.

Why

As I mentioned in #1010, when the game crashed unexpectedly, I had no way of knowing which recipes I had just looked up.

Effect

World Specific Same World Different World
No
WorldSpecificOff-SameWorld.mp4
WorldSpecificOff-DifferentWorld.mp4
Yes
WorldSpecific-SameWorld.mp4
WorldSpecific-DifferentWorld.mp4

Checklist

  • I have tested this PR in DevEnv
  • I have tested this PR in Fullpack(Daily 696)
  • This PR is in compliance with the GTNH AI Policy
  • This PR requires another PR in order to merge

@PlayfulPiano

Copy link
Copy Markdown

iirc bookmarks can similarly be wiped during crashes, unless that got fixed recently?

@vcwdfca

vcwdfca commented Aug 23, 2026

Copy link
Copy Markdown
Author

iirc bookmarks can similarly be wiped during crashes, unless that got fixed recently?

Bookmarks are saved to a file, so a crash is unlikely to erase them entirely. In practice, they remain fairly dependable most of the time.

As for the history log, it’s cleared upon each game startup. In my opinion, saving it would be both more practical and more dependable.

@slprime

slprime commented Aug 25, 2026

Copy link
Copy Markdown
Member

I'd recommend adding an option to disable saving history. I don't think saving history after a reload will be useful for everyone.

@vcwdfca

vcwdfca commented Aug 25, 2026

Copy link
Copy Markdown
Author

I'd recommend adding an option to disable saving history. I don't think saving history after a reload will be useful for everyone.

done.The default now falls back to the previous behavior.

@Dream-Master
Dream-Master requested a review from slprime August 26, 2026 04:03
@slprime
slprime merged commit bd595f7 into GTNewHorizons:master Aug 26, 2026
1 check passed
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.

[Suggestion]Save the history so that it persists between play sessions

3 participants