-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
Don't auto-generate the hidden field when creating profile stubs #10714
Conversation
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.
Meh, I'm okay with this. I liked it being there to try and inform users "hey, this hidden
field probably does something", but based on the plethora of issues we've gotten since then, clearly no one figures that out.
Hello @DHowett! Because this pull request has the p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (
|
…rosoft#10714) ## Summary of the Pull Request We no longer automatically write the 'hidden' field for profile stubs we create **Note**: This does not retroactively remove the automatically generated hidden fields in current settings files ## PR Checklist * [x] Closes microsoft#10539 * [x] CLA signed. If not, go over [here](https://cla.opensource.microsoft.com/microsoft/Terminal) and sign the CLA * [ ] Tests added/passed * [ ] Documentation updated. If checked, please file a pull request on [our docs repo](https://github.com/MicrosoftDocs/terminal) and link it here: #xxx * [ ] Schema updated. * [x] I work here ## Validation Steps Performed Deleted the ubuntu stub in my settings file, booted up terminal, new created stub did not have the hidden field. Created a fragment that overrides the hidden field and it worked.
) ## Summary of the Pull Request We no longer automatically write the 'hidden' field for profile stubs we create **Note**: This does not retroactively remove the automatically generated hidden fields in current settings files ## PR Checklist * [x] Closes #10539 * [x] CLA signed. If not, go over [here](https://cla.opensource.microsoft.com/microsoft/Terminal) and sign the CLA * [ ] Tests added/passed * [ ] Documentation updated. If checked, please file a pull request on [our docs repo](https://github.com/MicrosoftDocs/terminal) and link it here: #xxx * [ ] Schema updated. * [x] I work here ## Validation Steps Performed Deleted the ubuntu stub in my settings file, booted up terminal, new created stub did not have the hidden field. Created a fragment that overrides the hidden field and it worked.
Summary of the Pull Request
We no longer automatically write the 'hidden' field for profile stubs we create
Note: This does not retroactively remove the automatically generated hidden fields in current settings files
PR Checklist
Validation Steps Performed
Deleted the ubuntu stub in my settings file, booted up terminal, new created stub did not have the hidden field. Created a fragment that overrides the hidden field and it worked.