Skip to content

Document RemoteDesktopRegistrar.ConnectionCenterRequested event (ADO #562869)#2534

Draft
GrantMeStrength wants to merge 2 commits intodocsfrom
jken/562869-connectioncenterrequested-event
Draft

Document RemoteDesktopRegistrar.ConnectionCenterRequested event (ADO #562869)#2534
GrantMeStrength wants to merge 2 commits intodocsfrom
jken/562869-connectioncenterrequested-event

Conversation

@GrantMeStrength
Copy link
Copy Markdown
Contributor

Summary

New ref topic for the ConnectionCenterRequested event on RemoteDesktopRegistrar. This event notifies remote desktop provider apps (boot to cloud scenario) when the user requests the Connection Center.

Changes

  • New: remotedesktopregistrar_connectioncenterrequested.md — event ref topic
  • Updated: remotedesktopregistrar.md — added -events table linking to the new topic

Notes

  • Event handler signature (EventHandler<object>) needs confirmation from the spec: ConnectionCenterRequestedSpec.docx
  • LAF token requirement carried forward from existing class doc

Fixes ADO #562869

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new WinRT event reference topic for RemoteDesktopRegistrar.ConnectionCenterRequested and links it from the RemoteDesktopRegistrar class page, while also improving PackageManager.FindPackages* documentation to clarify scope and parameter format expectations.

Changes:

  • Added a new event reference topic: RemoteDesktopRegistrar.ConnectionCenterRequested.
  • Updated RemoteDesktopRegistrar to include an -events table linking to the new event.
  • Clarified PackageManager.FindPackages* docs (all-users scope + parameter format notes).

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
windows.system.remotedesktop.provider/remotedesktopregistrar.md Adds -events table and links to the new ConnectionCenterRequested event topic.
windows.system.remotedesktop.provider/remotedesktopregistrar_connectioncenterrequested.md New reference topic documenting the ConnectionCenterRequested event, including LAF guidance.
windows.management.deployment/packagemanager_findpackages_331793265.md Adds NOTE clarifying packageFamilyName format and common misinterpretations.
windows.management.deployment/packagemanager_findpackages_1597689551.md Expands remarks with all-users scope and adds NOTE clarifying packageName/packagePublisher formats.
windows.management.deployment/packagemanager_findpackages_1246181969.md Corrects/clarifies method description and adds all-users scope guidance in remarks.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread windows.management.deployment/packagemanager_findpackages_331793265.md Outdated
…#562869)

New event raised when the user requests the boot to cloud Connection Center
in a remote desktop provider app. Adds new event ref topic and updates the
RemoteDesktopRegistrar class overview with the -events table.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@GrantMeStrength GrantMeStrength force-pushed the jken/562869-connectioncenterrequested-event branch from fb10622 to cc55a2b Compare April 9, 2026 22:03
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@GrantMeStrength
Copy link
Copy Markdown
Contributor Author

Thanks for the review. Stripped the unrelated PackageManager.FindPackages commits (added by the Copilot coding agent) via force-push — the PR now contains only the two files for ADO #562869. The PackageManager improvements can go in a separate PR if they have merit.

The typo flag was valid but is now moot since those files are no longer in scope.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@GrantMeStrength
Copy link
Copy Markdown
Contributor Author

Waiting for Feature team.

@GrantMeStrength GrantMeStrength marked this pull request as draft April 10, 2026 20:35
- Description: 'Raised when the user requests...' -> 'Raised to notify
  a remote desktop provider app that the user has requested...'
  Clarifies this is a notification delivered to the app, not an in-app action.

- IMPORTANT note: 'RemoteDesktopRegistrar API' -> 'ConnectionCenterRequested event'
  The note is on the event page; referring to the parent class was ambiguous.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge PR is being processed and awaiting review/approval. docs-editor-pilot review-team-triage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants