-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
Serialize 'disabledProfileSources' #9038
Conversation
Just making sure - there aren't any OTHER settings we need to do this for, right? |
For whatever reason, |
"disabledProfileSources" is saved to `CascadiaSettings` _not_ `GlobalAppSettings` (and, even then, it's only read when it's used, never saved). This PR specifically detects if it was defined in settings.json, and copies it over when the settings are serialized. ## Validation Steps Performed 1. Added "disabledProfileSources" to settings.json, then serialized. --> "disabledProfileSources" is now maintained. 2. Updated `CascadiaSettings` serialization test Closes #9032 (cherry picked from commit 230fad5)
🎉 Handy links: |
Summary of the Pull Request
"disabledProfileSources" is saved to
CascadiaSettings
notGlobalAppSettings
(and, even then, it's only read when it's used, never saved). This PR specifically detects if it was defined in settings.json, and copies it over when the settings are serialized.References
#6800 - Settings UI epic
Validation Steps Performed
CascadiaSettings
serialization testCloses #9032