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

Transfer settings from Windows Registry to .NET Application Settings #100

Closed
3 of 4 tasks
gbakeman opened this issue Aug 10, 2023 · 5 comments · Fixed by #114
Closed
3 of 4 tasks

Transfer settings from Windows Registry to .NET Application Settings #100

gbakeman opened this issue Aug 10, 2023 · 5 comments · Fixed by #114
Assignees
Labels
enhancement New feature or request

Comments

@gbakeman
Copy link
Contributor

gbakeman commented Aug 10, 2023

WinNUT preferences are stored in the User Registry hive. This is a self-rolled solution that requires the user to venture into their Registry when required to access the preferences manually or clear them out. To take advantage of the more easily manageable and integrated system, we can use the .Net Application Settings system. This will be a multi-part project:

  • Modify the startup routine to check for existing registry entries. If they exist, run the migration wizard/dialog. At the end of the process, flip a setting indicating that the user was prompted before.
  • Modify all project references to the registry keys, and use the new settings system.
  • Remove the old 1.x INI import system (the last pre-release of WinNUT will be the last one to support it)
  • Remove the migration wizard and all code relating to the old Registry system #115

Registry/Preferences migration dialog

WinNUT will have added a new dialog (structured with new-to-WinNUT DataBinding, BackgroundWorker and MVC/MVP design patterns) that will assist the user with migrating to the standard .Net Application Settings system. References to the old system of registry preferences will be removed throughout WinNUT except as to assist the migration dialog. The migration dialog will support operations of importing, backing up (to .REG file) and deleting the old preference Registry keys. If the user opts to cancel or do nothing, then the old Registry keys will be left alone and a fresh copy of settings will be used instead. This dialog will only be displayed once as long as the new settings remain intact.

Please provide any feedback about this below.

@gbakeman gbakeman added the enhancement New feature or request label Aug 10, 2023
@gbakeman gbakeman added this to the 2.2 Stable Release milestone Aug 10, 2023
@gbakeman gbakeman self-assigned this Aug 10, 2023
@gbakeman gbakeman changed the title Transfer settings from Windows Registry to .NET Application Settings, redo Prefs_Gui Transfer settings from Windows Registry to .NET Application Settings Oct 5, 2023
@gbakeman gbakeman linked a pull request Oct 21, 2023 that will close this issue
11 tasks
@DUN-Cosmos
Copy link

It would be nice to add a backup & restore feature for the setting.

@gbakeman
Copy link
Contributor Author

gbakeman commented Oct 28, 2023

It would be nice to add a backup & restore feature for the setting.

Thank you for the comment. I'm definitely planning on an import and full backup system (which will likely create an importable .reg file), but I'm not so sure about a restore feature. I want this to be a one-and-done situation, so going forward WinNUT will rely on the new Settings system exclusively and the old Registry system will no longer exist in WinNUT. What do you think about this?

Edit; I also added more information about what I'm thinking in the original post if you wouldn't mind looking at that.

@DUN-Cosmos
Copy link

For the old Registry system, as you say a exported .reg file will be plainly sufficient.

I don't know how the .NET Application Settings work so I can't tall is this need a restore feature.

@gbakeman
Copy link
Contributor Author

For anyone monitoring this: please give the latest build a try. I believe all major work on this feature is completed. I've tested the import, backup and deletion features a few times on my own machine, but I'd like to hear from at least one other person confirming success. If I don't hear from anyone in a week or so, I think I'll go ahead and push out a pre-release.

@gbakeman gbakeman added the help wanted Extra attention is needed label Jan 31, 2024
@gbakeman gbakeman removed the help wanted Extra attention is needed label Feb 4, 2024
@gbakeman
Copy link
Contributor Author

gbakeman commented Feb 4, 2024

Going ahead and deploying.

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 a pull request may close this issue.

2 participants