Skip to content

RUM-11648: Fix LogEvent device types#2474

Merged
simaoseica-dd merged 3 commits into
developfrom
simaoseica/RUM-11648/fix-logevent-device-property
Sep 17, 2025
Merged

RUM-11648: Fix LogEvent device types#2474
simaoseica-dd merged 3 commits into
developfrom
simaoseica/RUM-11648/fix-logevent-device-property

Conversation

@simaoseica-dd

@simaoseica-dd simaoseica-dd commented Sep 16, 2025

Copy link
Copy Markdown
Contributor

What and why?

This PR aligns the Objective-C device type used in Logs with the centralized Swift Device model.
This ensures consistency across features and helps clients correctly use the Objective-C Logs APIs, such as the Kotlin Multiplatform SDK.

How?

It updates the Objective-C LogEvent.device and dd.device parameters to match the current Swift models.

Review checklist

  • Feature or bugfix MUST have appropriate tests (unit, integration)
  • Make sure each commit and the PR mention the Issue number or JIRA reference
  • Add CHANGELOG entry for user facing changes
  • Add Objective-C interface for public APIs (see our guidelines (internal) and run make api-surface)

@simaoseica-dd
simaoseica-dd requested review from a team as code owners September 16, 2025 12:58
0xnm
0xnm previously approved these changes Sep 16, 2025

@0xnm 0xnm left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm! need to update API surface file as well

@simaoseica-dd
simaoseica-dd merged commit 8f4ba54 into develop Sep 17, 2025
17 checks passed
@simaoseica-dd
simaoseica-dd deleted the simaoseica/RUM-11648/fix-logevent-device-property branch September 17, 2025 08:30
@simaoseica-dd simaoseica-dd mentioned this pull request Sep 18, 2025
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.

4 participants