-
Notifications
You must be signed in to change notification settings - Fork 8.5k
[Enterprise, GPO] Add "Default Terminal app" policy to definition template #18363
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
Conversation
Can you please assign me. |
/azp run |
Commenter does not have sufficient privileges for PR 18363 in repo microsoft/terminal |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
So, this is technically a third-party extensibility point. Can the policy support a freeform GUID entry (or GUID pair entry, as it were) for any third party terminals that have onboarded? |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
@DHowett But while testing I found a strange and uncool behavior: Even if I define a "disabledList", the values will be deleted in Registry if the policy is disabled. Is this a blocking problem for us? (I have tested that WT and Console won't crash.) Solutions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for doing this! I understand your comments about the free entry field, I'll need to think about them for a bit. This is looking very good.
@DHowett |
b1fce51
to
36b7c89
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alright, I thought about this for a while. Clearly, it has been a while.
We have no other 3p terminals that plug in to this mechanism.
Your solution of having the dropdown is, right now, more elegant and closer to product truth.
I think your original implementation was the right one.
Thank you so much for doing it, and for your patience. :)
I reset your branch to the point in time before you did the freeform textbox work. :) |
@DHowett |
I got pulled into a meeting before I could click the button :P |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Thanks for doing this!
@DHowett , @carlos-zamora Btw, please don't forget the docs merge on release. |
I can't tell you that until I've minted the release (considering that it contains the day of the year in the version number.) It'll be included with the next servicing updates. Thanks for the reminder about docs. |
…plate (#18363) This PR adds a new policy definition to the ADMX templates for settings the default Terminal application in Windows. > [!Note] > This PR does not change any code of Windows, Console Host or Windows Terminal. It only adds the definition for a new policy to the templates. I got the registry values form the documentation and by testing the values. The policy is only available as user policy because the registry values have to be in HKCU. The Policy is implemented as preference (not inside the Policy key) and therefore keeps it's value on removing (not configured) it. You can see this in `gpedit.msc` on the policy symbol and the hint in the description. Closes #18302 Refs #18303 (cherry picked from commit 68d9e0d) Service-Card-Id: PVTI_lADOAF3p4s4AmhmQzgZlw44 Service-Version: 1.22
…plate (#18363) This PR adds a new policy definition to the ADMX templates for settings the default Terminal application in Windows. > [!Note] > This PR does not change any code of Windows, Console Host or Windows Terminal. It only adds the definition for a new policy to the templates. I got the registry values form the documentation and by testing the values. The policy is only available as user policy because the registry values have to be in HKCU. The Policy is implemented as preference (not inside the Policy key) and therefore keeps it's value on removing (not configured) it. You can see this in `gpedit.msc` on the policy symbol and the hint in the description. Closes #18302 Refs #18303 (cherry picked from commit 68d9e0d) Service-Card-Id: PVTI_lADOAF3p4s4AxadtzgZlw40 Service-Version: 1.23
Summary of the Pull Request
This PR implements a policy definition for setting/enforcing the default terminal application.
References and Relevant Issues
GitHub (issues):
Docs:
Detailed Description of the Pull Request / Additional comments
This PR adds a new policy definition to the ADMX templates for settings the default Terminal application in Windows.
Note
This PR does not change any code of Windows, Console Host or WIndows Terminal. It only adds the definiton for a new policy to the templates.
I got the registry values form the documentation and by testing the values.
The policy is only available as user policy because the registry values have to be in HKCU.
The Policy is implemented as preference (not inside the Policy key) and therefore keeps it's value on removing (not configured) it. You can see this in

gpedit.msc
on the policy symbol and the hint in the description.Validation Steps Performed
Manual testing the policy.
PR Checklist