Skip to content

Conversation

@cdn34dd
Copy link
Contributor

@cdn34dd cdn34dd commented Jan 12, 2026

What does this PR do?

Changes the format of the View key format sent to Datadog used my react-navigation to match the standard view format used by other mobile SDKs. This does not change any of the internal functionality of the SDK, it simply changes the format of the view key that is sent.

Aside from the key format correction, this change also fixes an issue with url_path_group that was getting incorrectly calculated due to the wrong format.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)
  • If this PR is auto-generated, please make sure also to manually update the code related to the change

@cdn34dd cdn34dd requested a review from a team as a code owner January 12, 2026 17:10
@datadog-official
Copy link

datadog-official bot commented Jan 12, 2026

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 4e8f39e | Docs | Datadog PR Page | Was this helpful? Give us feedback!

@cdn34dd cdn34dd force-pushed the carlosnogueira/RUM-13689/update-view-key-format branch from 4931487 to 2f3a1cb Compare January 12, 2026 17:58
sbarrio
sbarrio previously approved these changes Jan 13, 2026
@cdn34dd cdn34dd force-pushed the carlosnogueira/RUM-13689/update-view-key-format branch from 2f3a1cb to 4e8f39e Compare January 14, 2026 09:53
@cdn34dd cdn34dd merged commit 84230b6 into feature/v3 Jan 14, 2026
9 checks passed
@cdn34dd cdn34dd deleted the carlosnogueira/RUM-13689/update-view-key-format branch January 14, 2026 10:25
@sbarrio sbarrio mentioned this pull request Jan 22, 2026
4 tasks
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.

3 participants