Incorrect type in settings schema for startingDirectory property #18762

Closed
opened 2026-01-31 06:23:37 +00:00 by claunia · 4 comments
Owner

Originally created by @PetSerAl on GitHub (Oct 28, 2022).

In settings schema (https://aka.ms/terminal-profiles-schema) type of startingDirectory property described as string. But when you set inherit starting directory from parent process, the null value get saved to the property.

Originally created by @PetSerAl on GitHub (Oct 28, 2022). In settings schema (https://aka.ms/terminal-profiles-schema) type of `startingDirectory` property described as `string`. But when you set inherit starting directory from parent process, the `null` value get saved to the property.
Author
Owner

@zadjii-msft commented on GitHub (Oct 28, 2022):

Huh. Didn't know that null wasn't a valid value for a string in json schema. See: https://stackoverflow.com/questions/16241333/specify-a-value-can-be-a-string-or-null-with-json-schema

@zadjii-msft commented on GitHub (Oct 28, 2022): Huh. Didn't know that `null` wasn't a valid value for a string in json schema. See: https://stackoverflow.com/questions/16241333/specify-a-value-can-be-a-string-or-null-with-json-schema
Author
Owner

@BenConstable9 commented on GitHub (Nov 18, 2022):

Happy to submit a PR for this simple fix.

@BenConstable9 commented on GitHub (Nov 18, 2022): Happy to submit a PR for this simple fix.
Author
Owner

@ghost commented on GitHub (Dec 14, 2022):

:tada:This issue was addressed in #14408, which has now been successfully released as Windows Terminal v1.15.3465.0 and v1.15.3466.0.🎉

Handy links:

@ghost commented on GitHub (Dec 14, 2022): :tada:This issue was addressed in #14408, which has now been successfully released as `Windows Terminal v1.15.3465.0 and v1.15.3466.0`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v1.15.3466.0) * [Store Download](https://www.microsoft.com/store/apps/9n8g5rfz9xk3?cid=storebadge&ocid=badge)
Author
Owner

@ghost commented on GitHub (Dec 14, 2022):

:tada:This issue was addressed in #14408, which has now been successfully released as Windows Terminal Preview v1.16.3463.0 and v1.16.3464.0.🎉

Handy links:

@ghost commented on GitHub (Dec 14, 2022): :tada:This issue was addressed in #14408, which has now been successfully released as `Windows Terminal Preview v1.16.3463.0 and v1.16.3464.0`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v1.16.3464.0) * [Store Download](https://www.microsoft.com/store/apps/9n8g5rfz9xk3?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#18762