Skip to content

Revert "RDKEMW-999 : Add COM-RPC support to ResourceManager plugin"#741

Closed
naveen-0206 wants to merge 1 commit intodevelopfrom
revert-701-feature/8_jan_apis_RM
Closed

Revert "RDKEMW-999 : Add COM-RPC support to ResourceManager plugin"#741
naveen-0206 wants to merge 1 commit intodevelopfrom
revert-701-feature/8_jan_apis_RM

Conversation

@naveen-0206
Copy link
Contributor

Reverts #701

@naveen-0206 naveen-0206 requested a review from a team as a code owner February 20, 2026 12:58
Copilot AI review requested due to automatic review settings February 20, 2026 12:58
Copy link
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

This PR reverts PR #701, which had added COM-RPC support to the ResourceManager plugin. The revert cleanly removes all artifacts of the COM-RPC implementation while preserving the existing ResourceManager plugin documentation (ResourceManagerPlugin.md).

Changes:

  • Removed the COM-RPC interface definition for ResourceManager
  • Removed the generated API documentation for the COM-RPC interface
  • Removed the interface ID constant from the central ID registry
  • Removed the documentation sidebar entry for the COM-RPC version

Reviewed changes

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

File Description
docs/apis/ResourceManager.md Removed auto-generated documentation for COM-RPC ResourceManager interface
docs/_sidebar.md Removed sidebar entry for ResourceManager.md (keeping ResourceManagerPlugin.md)
apis/ResourceManager/IResourceManager.h Removed COM-RPC interface header definition
apis/Ids.h Removed ID_RESOURCEMANAGER constant from the ID registry

@naveen-0206
Copy link
Contributor Author

No need to revert the changes i have made a fix on the top of earlier changes

@github-actions github-actions bot locked and limited conversation to collaborators Feb 23, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants