Skip to content

Add reset cache and settings buttons to SUI #18907

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

Merged
merged 1 commit into from
May 14, 2025

Conversation

carlos-zamora
Copy link
Member

Summary of the Pull Request

Adds the ability to reset the settings.json file and application state via the Settings UI. Since these are destructive operations, the destructive styling is applied, as well as a confirmation prompt notifying the user that this occurs immediately.

Validation Steps Performed

✅ "reset settings" results in the correct contents for settings.json and state.json
✅ "reset cache" results in the correct contents for state.json

Closes #947

@microsoft-github-policy-service microsoft-github-policy-service bot added Issue-Feature Complex enough to require an in depth planning process and actual budgeted, scheduled work. Area-SettingsUI Anything specific to the SUI Product-Terminal The new Windows Terminal. labels May 13, 2025
@carlos-zamora
Copy link
Member Author

Demo

image
image
image

Open to feedback on word choice

CC @nguyen-dows

@nguyen-dows
Copy link
Contributor

Wording LGTM!

@carlos-zamora carlos-zamora merged commit 9e0ca3a into main May 14, 2025
19 checks passed
@carlos-zamora carlos-zamora deleted the dev/cazamor/sui/reset-settings branch May 14, 2025 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-SettingsUI Anything specific to the SUI Issue-Feature Complex enough to require an in depth planning process and actual budgeted, scheduled work. Product-Terminal The new Windows Terminal.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Settings UI] Add a button to reset the settings to defaults
4 participants