Skip to content

[RUM-12388] [V3] Support SVG in Session Replay#1026

Closed
cdn34dd wants to merge 1 commit into
feature/v3from
carlosnogueira/RUM-12388/add-svg-sr-support-v3
Closed

[RUM-12388] [V3] Support SVG in Session Replay#1026
cdn34dd wants to merge 1 commit into
feature/v3from
carlosnogueira/RUM-12388/add-svg-sr-support-v3

Conversation

@cdn34dd

@cdn34dd cdn34dd commented Oct 24, 2025

Copy link
Copy Markdown
Contributor

What does this PR do?

This PR adds support for tracking SVG images in React Native Session Replay.

Same as this PR: #985

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 review from a team as code owners October 24, 2025 10:55

@maycmlee maycmlee left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just a small suggestion, but approving

yarn android
```

The `datadog-generate-sr-assets` CLI utility scans your codebase for SVG elements and pre-generates optimized assets that will be included in your native builds.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
The `datadog-generate-sr-assets` CLI utility scans your codebase for SVG elements and pre-generates optimized assets that will be included in your native builds.
The `datadog-generate-sr-assets` CLI utility scans your codebase for SVG elements and pre-generates optimized assets that are included in your native builds.

@sbarrio

sbarrio commented Oct 27, 2025

Copy link
Copy Markdown
Contributor

Let's not forget to also update the Podfile.lock for the benchmarks app 🙏

Also, won;t we have to replicate the mimeType change for iOS v3 as well?, I see dd-sdk-android rolled out a 3.2.0 including it (same as for v2), but iOS apparently hasn't done that yet, right?

@sbarrio

sbarrio commented Oct 29, 2025

Copy link
Copy Markdown
Contributor

FYI: I merged the PR updating feature/v3 with the latest changes from develop, so this PR may not be necessary anymore.

@cdn34dd cdn34dd closed this Nov 21, 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.

3 participants