RDKEMW-9595: IAnalytics: Added additionalContext to sendEvent #582
Conversation
Reason for change: additionalContex for Analytics.sendEvent Test Procedure: Build and use new param Risks: Low Version: Patch Signed-off-by: Adrian Muzyka <[email protected]>
adrianM27
force-pushed
the
topic/RDKEMW-9595_Analytics_addtionalContext
branch
from
1cc1995 to
018d532
Compare
There was a problem hiding this comment.
Pull Request Overview
This PR adds a new additionalContext parameter to the Analytics.sendEvent API to allow passing additional context information in JSON format as an opaque string without schema validation.
Key Changes:
- Added
additionalContextparameter to theSendEventmethod signature in the IAnalytics interface - Updated JSON schema definition to include the new parameter
- Updated API documentation with parameter descriptions and examples
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| apis/Analytics/IAnalytics.h | Added additionalContext parameter to SendEvent method signature with @opaque annotation |
| tools/json_generator/output/Analytics/Analytics.json | Added additionalContext definition in the schema with description and example |
| docs/apis/AnalyticsPlugin.md | Updated parameter table and example to include the new optional additionalContext parameter; also includes unrelated configuration section updates |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 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.
Reason for change: additionalContex for Analytics.sendEvent
Test Procedure: Build and use new param
Risks: Low
Version: Patch