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

Warn user about other instances owning the settings file. #135

Merged
merged 1 commit into from
May 12, 2024

Conversation

cristian64
Copy link
Collaborator

When multiple DME instances are created, only the first one gets to own the settings file now; previously, the last DME instance to close would rewrite the settings file, potentially causing data loss from the previous instances.

A warning dialog will be shown on startup if the settings lockfile cannot be acquired. On exit, if there are unsaved changes to the watch list, the user will be given an opportunity to save it into a separate .dmw file.

When multiple DME instances are created, only the first one gets to
own the settings file now; previously, the last DME instance to close
would rewrite the settings file, potentially causing data loss from the
previous instances.

A warning dialog will be shown on startup if the settings lockfile
cannot be acquired. On exit, if there are unsaved changes to the watch
list, the user will be given an opportunity to save it into a separate
`.dmw` file.
Copy link
Collaborator

@dreamsyntax dreamsyntax left a comment

Choose a reason for hiding this comment

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

Tested on Windows post rebase.

@dreamsyntax dreamsyntax merged commit c416d6b into aldelaro5:master May 12, 2024
4 checks 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.

2 participants