Skip to content

Added a note about MSStoreProductIdentifier non-support to RestInterface_1_1.cpp. - #6458

Closed
Imre Eilertsen (DandelionSprout) wants to merge 1 commit into
microsoft:masterfrom
DandelionSprout:patch-2
Closed

Added a note about MSStoreProductIdentifier non-support to RestInterface_1_1.cpp.#6458
Imre Eilertsen (DandelionSprout) wants to merge 1 commit into
microsoft:masterfrom
DandelionSprout:patch-2

Conversation

@DandelionSprout

@DandelionSprout Imre Eilertsen (DandelionSprout) commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

📖 Description

The following section in https://github.com/microsoft/winget-cli/blob/master/src/AppInstallerCLITests/RestInterface_1_1.cpp claims that the following would be able to get working in a manifest:

(...)
{
  "Architecture": "x64",
  "InstallerType": "msstore",
  "MSStoreProductIdentifier": "9nblggh4nns1"
}
(...)

It can't.

So this PR adds a note about that it can't.

🔗 References

🔍 Validation

Added a secondary installer to a manifest that used the syntaxing used for the msstore InstallerType in https://github.com/microsoft/winget-cli/blob/master/src/AppInstallerCLITests/RestInterface_1_1.cpp

✅ Checklist

📋 Issue Type

  • Bug fix
  • Feature
  • Task
Microsoft Reviewers: Open in CodeFlow

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@DandelionSprout

Copy link
Copy Markdown
Contributor Author

Closing in light of microsoft/winget-pkgs#416221 indicating it is supported (sort-of) after all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant