Skip to content
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

[YamlCreate] Request ARP Values on Quick Update #129471

Merged
merged 3 commits into from
Dec 6, 2023
Merged

Conversation

Trenly
Copy link
Contributor

@Trenly Trenly commented Dec 6, 2023

Currently, YamlCreate ignores AppsAndFeaturesEntries entirely. This PR begins the work of adding support for them.
The support is currently limited to applications with Existing ARP entries and only in the Quick Update flow.
The support also skips ProductCode and UpgradeCode as there is complexity around those fields related to when they have been fetched automatically from the updated application.

If there are entries that exist, the script will now prompt users to validate that the entry is correct. Providing no response to the prompt will keep the entry as it was.

image

cc @mdanish-kh

Microsoft Reviewers: Open in CodeFlow

@wingetbot
Copy link
Collaborator

Service Badge  Service Badge  

@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Attention This work item needs to be reviewed by a member of the core team. label Dec 6, 2023
Tools/YamlCreate.ps1 Show resolved Hide resolved
Tools/YamlCreate.ps1 Show resolved Hide resolved
@microsoft-github-policy-service microsoft-github-policy-service bot removed the Needs-Attention This work item needs to be reviewed by a member of the core team. label Dec 6, 2023
@microsoft-github-policy-service microsoft-github-policy-service bot added the Moderator-Approved One of the Moderators has reviewed and approved this PR label Dec 6, 2023
@denelon denelon merged commit 0d723bf into microsoft:master Dec 6, 2023
1 check passed
@Trenly Trenly deleted the ARP branch December 6, 2023 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Moderator-Approved One of the Moderators has reviewed and approved this PR Project-File
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants