ignore json profile data not understood #2648

Closed
opened 2026-01-30 23:01:10 +00:00 by claunia · 2 comments
Owner

Originally created by @fredless on GitHub (Jul 8, 2019).

Originally assigned to: @zadjii-msft on GitHub.

Summary of the new feature/enhancement

Current implementation seems to clean up json parameters entries in the config file not understood or implemented. Rather than removing unknown parameters, perhaps just ignore them.

Proposed technical implementation details (optional)

Moving\syncing profiles.json between profiles\machines with differing builds (i.e. msftstore release versus "nightly") can result in undesired configuration removal. If you are 2-way syncing profile data it's even worse: configuration removal would head back to the source machine, where the configuration was valid.

Originally created by @fredless on GitHub (Jul 8, 2019). Originally assigned to: @zadjii-msft on GitHub. # Summary of the new feature/enhancement Current implementation seems to clean up json ~~parameters~~ *entries in the config file* not understood or implemented. Rather than removing unknown parameters, perhaps just ignore them. # Proposed technical implementation details (optional) Moving\syncing profiles.json between profiles\machines with differing builds (i.e. msftstore release versus "nightly") can result in undesired configuration removal. If you are 2-way syncing profile data it's even worse: configuration removal would head back to the source machine, where the configuration was valid.
claunia added the Area-SettingsIssue-TaskNeeds-Tag-FixProduct-Terminal labels 2026-01-30 23:01:10 +00:00
Author
Owner

@zadjii-msft commented on GitHub (Jul 11, 2019):

This is something that will be fixed by the work that needs to be done for #754. I'll parent it to that deliverable.

@zadjii-msft commented on GitHub (Jul 11, 2019): This is something that _will_ be fixed by the work that needs to be done for #754. I'll parent it to that deliverable.
Author
Owner

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

This is done as of #(the pull request that disabled settings reserialization), actually.

@DHowett-MSFT commented on GitHub (Sep 19, 2019): This is done as of #(the pull request that disabled settings reserialization), actually.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#2648