Skip to content

Rewrite backup system - #1299

Merged
OverloadedOrama merged 6 commits into
Orama-Interactive:masterfrom
Variable-ind:rework-backup-system
Jul 9, 2025
Merged

OverloadedOrama merged 6 commits into
Orama-Interactive:masterfrom
Variable-ind:rework-backup-system

Conversation

@Variable-ind

@Variable-ind Variable-ind commented Jul 6, 2025

Copy link
Copy Markdown
Contributor

The new system keeps backups of upto 20 last sessions (older session backups get removed)
This is basically the LifeSaver extension, integrated to core.

(some other related fixes i found while integrating):

  • fixed Global.popup_error(tr("File failed to save. Error code %s (%s)") % [err, error_string(err)]) accidentally placed after return instead of before.
  • removed duplicate occurrences of "user://backups" to a single variable (which should be less prone to bugs)

@OverloadedOrama OverloadedOrama left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thank you!

@OverloadedOrama
OverloadedOrama merged commit a434510 into Orama-Interactive:master Jul 9, 2025
@Variable-ind
Variable-ind deleted the rework-backup-system branch July 11, 2025 04:45
OverloadedOrama added a commit that referenced this pull request Jul 15, 2025
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.

2 participants