WT should inject GUIDs for guidless custom profiles #4019

Closed
opened 2026-01-30 23:35:59 +00:00 by claunia · 5 comments
Owner

Originally created by @dev-nicolaos on GitHub (Sep 24, 2019).

Environment

Windows build number: 10.0.18362.0
Windows Terminal version: 0.5.2661.0

Steps to reproduce

  1. Open WT
  2. Open profiles.json
  3. Create an object in the profiles array
  4. Fill in some fields such as name, commandline, startingDirectory, colorScheme, etc.
  5. Save profiles.json

Expected behavior

WT will generate/add a guid to the new custom profile, that way it can be set as the defaultProfile. This was the case in previous versions of WT, before separation of cascading settings was implemented.

Actual behavior

WT does not generate/add a guid to the new profile, thus there appears to be no way to set your custom profile as the default.

Originally created by @dev-nicolaos on GitHub (Sep 24, 2019). <!-- 🚨🚨🚨🚨🚨🚨🚨🚨🚨🚨 I ACKNOWLEDGE THE FOLLOWING BEFORE PROCEEDING: 1. If I delete this entire template and go my own path, the core team may close my issue without further explanation or engagement. 2. If I list multiple bugs/concerns in this one issue, the core team may close my issue without further explanation or engagement. 3. If I write an issue that has many duplicates, the core team may close my issue without further explanation or engagement (and without necessarily spending time to find the exact duplicate ID number). 4. If I leave the title incomplete when filing the issue, the core team may close my issue without further explanation or engagement. 5. If I file something completely blank in the body, the core team may close my issue without further explanation or engagement. All good? Then proceed! --> <!-- This bug tracker is monitored by Windows Terminal development team and other technical folks. **Important: When reporting BSODs or security issues, DO NOT attach memory dumps, logs, or traces to Github issues**. Instead, send dumps/traces to secure@microsoft.com, referencing this GitHub issue. If this is an application crash, please also provide a Feedback Hub submission link so we can find your diagnostic data on the backend. Use the category "Apps > Windows Terminal (Preview)" and choose "Share My Feedback" after submission to get the link. Please use this form and describe your issue, concisely but precisely, with as much detail as possible. --> # Environment ```none Windows build number: 10.0.18362.0 Windows Terminal version: 0.5.2661.0 ``` # Steps to reproduce 1. Open WT 1. Open profiles.json 1. Create an object in the `profiles` array 1. Fill in some fields such as `name`, `commandline`, `startingDirectory`, `colorScheme`, etc. 1. Save profiles.json # Expected behavior WT will generate/add a guid to the new custom profile, that way it can be set as the `defaultProfile`. This was the case in previous versions of WT, before separation of cascading settings was implemented. # Actual behavior WT does not generate/add a guid to the new profile, thus there appears to be no way to set your custom profile as the default.
Author
Owner

@DHowett-MSFT commented on GitHub (Sep 24, 2019):

Curious. Thanks for the catch! For now, if you generate a guid and set it for that guid-less profile you can set that as a default.
Sorry for the miss.

@DHowett-MSFT commented on GitHub (Sep 24, 2019): Curious. Thanks for the catch! For now, if you generate a guid and set it for that guid-less profile you can set _that_ as a default. Sorry for the miss.
Author
Owner

@DHowett-MSFT commented on GitHub (Sep 24, 2019):

This probably came about in 0.4 when we stopped writing folks' settings files back out to disk, honestly.

@DHowett-MSFT commented on GitHub (Sep 24, 2019): This probably came about in 0.4 when we stopped writing folks' settings files back out to disk, honestly.
Author
Owner

@DHowett-MSFT commented on GitHub (Oct 8, 2019):

I've converted this into a task for injecting the GUID into the guidless profile. Thanks for reporting!

@DHowett-MSFT commented on GitHub (Oct 8, 2019): I've converted this into a task for injecting the GUID into the guidless profile. Thanks for reporting!
Author
Owner

@DHowett-MSFT commented on GitHub (May 7, 2020):

Continuing my inability to decide about this bug, I think it's better if we simply move away from GUIDs for user-facing fixtures as much as possible. They're great internal identifier. They're awful external ones.

I'm going to mark this, the incidental issue that this issue actually implies ("i can't set default a guid-less profile"), as fixed by #5706.

@DHowett-MSFT commented on GitHub (May 7, 2020): Continuing my inability to decide about this bug, I think it's better if we simply move away from GUIDs for user-facing fixtures as much as _possible_. They're great internal identifier. They're awful external ones. I'm going to mark this, the incidental issue that this issue actually implies ("i can't set default a guid-less profile"), as fixed by #5706.
Author
Owner

@ghost commented on GitHub (Jun 18, 2020):

:tada:This issue was addressed in #5706, which has now been successfully released as Windows Terminal Preview v1.1.1671.0.🎉

Handy links:

@ghost commented on GitHub (Jun 18, 2020): :tada:This issue was addressed in #5706, which has now been successfully released as `Windows Terminal Preview v1.1.1671.0`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v1.1.1671.0) * [Store Download](https://www.microsoft.com/store/apps/9n0dx20hk701?cid=storebadge&ocid=badge)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#4019