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

Add settings reset & advanced section #1154

Merged
merged 22 commits into from
Oct 9, 2024

Conversation

JovannMC
Copy link
Contributor

@JovannMC JovannMC commented Sep 6, 2024

Create advanced section containing some config functions - settings reset for GUI/server/all and opening SlimeVR's data folder.

Fixes #926, requires SlimeVR/SolarXR-Protocol#146.

image

Done so we don't have to maintain two sets of "default" configs by copying the properties from defaultGUIConfig (while excluding "lang" and "doneOnboarding") and manually re-adding defaults for "devSettings.

Both cause eslint/ts errors but are intentionally ignored because it works lol.
Make opening data folder (renamed from config to data) work, changed from grid to rows, removed unneeded code, add comment to InterfaceSettings
gui/public/i18n/en/translation.ftl Outdated Show resolved Hide resolved
gui/src-tauri/src/main.rs Outdated Show resolved Hide resolved
gui/src/components/settings/pages/AdvancedSettings.tsx Outdated Show resolved Hide resolved
gui/src/components/settings/pages/AdvancedSettings.tsx Outdated Show resolved Hide resolved
gui/src-tauri/tauri.conf.json Show resolved Hide resolved
gui/src/components/settings/pages/InterfaceSettings.tsx Outdated Show resolved Hide resolved
gui/src/components/settings/pages/AdvancedSettings.tsx Outdated Show resolved Hide resolved
JovannMC and others added 3 commits September 7, 2024 16:27
Add variable for the "type" of reset being done, use Tauri's appConfigDir(), use bug icon, change gap and comment

Co-Authored-By: Uriel <imurx@proton.me>
my bad x2
Co-Authored-By: Uriel <imurx@proton.me>
@JovannMC JovannMC requested a review from ImUrX September 8, 2024 15:15
Copy link
Member

@ImUrX ImUrX left a comment

Choose a reason for hiding this comment

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

just use all as default, no need for an unknown in here

gui/public/i18n/en/translation.ftl Outdated Show resolved Hide resolved
Co-authored-by: Uriel <imurx@proton.me>
@ButterscotchV ButterscotchV added Priority: Normal The default priority Area: GUI Related to the GUI Type: Enhancement Adds or improves a feature labels Sep 17, 2024
@ButterscotchV ButterscotchV added Area: Server Related to the server Area: Translation Improvements or additions to translations labels Sep 17, 2024
@ButterscotchV ButterscotchV mentioned this pull request Sep 29, 2024
Copy link
Member

@ButterscotchV ButterscotchV left a comment

Choose a reason for hiding this comment

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

🐛

gui/src/components/settings/pages/AdvancedSettings.tsx Outdated Show resolved Hide resolved
Copy link
Member

@ButterscotchV ButterscotchV left a comment

Choose a reason for hiding this comment

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

oops, check my comment 🐛 (sad bug)

JovannMC and others added 2 commits October 8, 2024 16:17
Co-Authored-By: Butterscotch! <5095026+ButterscotchV@users.noreply.github.com>
@ImUrX ImUrX requested a review from ButterscotchV October 9, 2024 01:06
Copy link
Member

@ButterscotchV ButterscotchV left a comment

Choose a reason for hiding this comment

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

🐄

@loucass003 loucass003 merged commit a2d842d into SlimeVR:main Oct 9, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: GUI Related to the GUI Area: Server Related to the server Area: Translation Improvements or additions to translations Priority: Normal The default priority Type: Enhancement Adds or improves a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature request] Reset settings to default
4 participants