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

Feature: Autosave/Autoload from Path #121

Open
dreamsyntax opened this issue Apr 26, 2024 · 3 comments
Open

Feature: Autosave/Autoload from Path #121

dreamsyntax opened this issue Apr 26, 2024 · 3 comments

Comments

@dreamsyntax
Copy link
Collaborator

While the automatic saving/loading of the watchlist from a temp file in appdata is useful, I find myself losing work in a multi user dmw if I forget to save.

  • allow auto load/save on close to use a last opened/saved file (only when a persistant setting is set)
  • prompt on closing? If temp file is being used.
@cristian64
Copy link
Collaborator

cristian64 commented May 12, 2024

As of #135, users will at least be warned, and will have a chance to save the file on exit if it has unsaved changes.

EDIT: I had misunderstood this issue; #135 was rather about warning when multiple DME instances were running.

@dreamsyntax
Copy link
Collaborator Author

#135 resolves issues with multiple DME instances - now merged.

Attached in a below use case I want to address before we can close this issue.

  1. shared git repo with DMWs in it
  2. open the DMW directly with DME, make changes. Save and quit. Commit changes. A day goes by, other people are working on the same DMW and add updates.
  3. pull from git repo as someone updated it.
  4. Later open DME, and since it auto keeps the changes from last DMW load, I forget to re-load from the file in the git repo (thus losing the changes from the other user's commits)

Effectively, I'm looking for an auto read from specific file directory mode rather than saving to/from the dolphin-memory-engine.ini.

@cristian64
Copy link
Collaborator

In some offline discussion, these actions can be added to the File menu:

File > [x] Autoload Last File
     > [x] Autosave Last File

C: Second option only enabled if first option is enabled, perhaps?
D: I like it!
C: If only first option is enabled, the prompt is shown.
D: Yeah, that sounds ideal.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants