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

FF8: Update metadata file when a save file is created #655

Merged

Conversation

myst6re
Copy link
Contributor

@myst6re myst6re commented Feb 14, 2024

Summary

Rewrite metadata file when a save file is created to prevent save file loss.

Currently the metadata file is updated when the game is closed. Some users experienced save file loss, maybe because the metadata file was not updated correctly and the ff8 launcher removed the save files which were detected as invalid.

Motivation

Losing your save is very serious situation, I never finished the game Child Of Light because Ubisoft devs do not understand the concept of redundancy and do not let you create several saves. Crash, lost hours, cry a lot, bargain, accept.

ACKs

  • I have updated the Changelog.md file
  • I did test my code on FF7
  • I did test my code on FF8

@myst6re myst6re force-pushed the fix/ff8-prevent-lost-save branch 2 times, most recently from 6be58ad to 2682497 Compare February 14, 2024 22:45
@myst6re
Copy link
Contributor Author

myst6re commented Feb 14, 2024

Also added a patch for a faster CI on forks, see succesful build here: https://github.com/myst6re/FFNx/actions/runs/7908320981/job/21587237081

@myst6re myst6re marked this pull request as ready for review February 14, 2024 22:46
@julianxhokaxhiu julianxhokaxhiu added the enhancement New feature or request label Feb 15, 2024
@julianxhokaxhiu julianxhokaxhiu added this to the 1.18.0 milestone Feb 15, 2024
@julianxhokaxhiu
Copy link
Owner

Something is weird, not sure why but despite installing CMake 3.28.3, the version 3.27.9 is detected while this was not happening before. I have to find out why, but I'll work on this later. Thanks for now, LGTM!

@julianxhokaxhiu julianxhokaxhiu changed the title FF8 Steam: Rewrite metadata file when a save file is created to prevent save file loss FF8: Update metadata file when a save file is created Feb 15, 2024
@julianxhokaxhiu julianxhokaxhiu merged commit 405acdc into julianxhokaxhiu:master Feb 15, 2024
@julianxhokaxhiu
Copy link
Owner

Thank you!

@myst6re myst6re deleted the fix/ff8-prevent-lost-save branch February 15, 2024 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants