-
Notifications
You must be signed in to change notification settings - Fork 8.4k
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
Stop roaming settings for now #1770
Comments
Usually, on GNU/Linux, the settings go in For a while, I have put all physical files for Windows Terminal settings in I don't know your technical reasons, but for the POV of the user, Thanks. |
This is biting me already where I have different |
Just my €0.02: IMVHO this has higher priority than #1258 (cascading settings Defaults -> User). Of course, the full settings experience needs both. |
Also migrate existing settings from RoamingState to LocalState. Fixes #1770.
* Stop Roaming settings Also migrate existing settings from RoamingState to LocalState. Fixes #1770. * * de-dupe these functions * const a pair of things * This should be in the previous commit * use `unique_hfile`'s * Make some of these wil things cleaner
🎉This issue was addressed in #2298, which has now been successfully released as Handy links: |
With the new preview version v0.4, settings sync has stopped, how should I sync my settings? I use a symbolic link which link profiles.json in Dropbox, but modifications to this file are not effective immediately, I need to restart Windows Terminal to take the effect. Where am I wrong? THX |
Settings sync was probably never working for you because of that symbolic link. For now, you're going to have to keep relaunching Windows Terminal whenever the settings in Dropbox change, because we cannot detect when that file changes without reading your entire dropbox directory. That's a lot more work than anybody wants Terminal to do. |
Thanks for your reply. |
Until we come up with a good story for settings overrides and default profiles on different machines, we're going to want to move the settings from
RoamingState
toLocalState
. Since right now a user's settings include everything we know about them for a specific machine, we're not capable of safely roaming them.Roaming machine-specific settings causes (but is not limited to causing) the following issues:
We should make the move to localstate sooner rather than later, since we only just released v0.2. This is still going to be a minor breaking change, but we should include some migration logic.
That migration logic should use a file move (instead of a read-parse-write cycle), as some folks are using symbolic links to manage their settings.
Refer:
The text was updated successfully, but these errors were encountered: