Skip to content

✨ [RUM-14582] track view counters#21

Merged
bcaudan merged 4 commits into
mainfrom
bcaudan/main-view-part2
Mar 2, 2026
Merged

✨ [RUM-14582] track view counters#21
bcaudan merged 4 commits into
mainfrom
bcaudan/main-view-part2

Conversation

@bcaudan

@bcaudan bcaudan commented Feb 19, 2026

Copy link
Copy Markdown
Collaborator

Motivation

Add counts of error, action and resource related to the main process view

Changes

  • Improve types around assembly
  • Listen to ServerEvents to populate event counters
  • Throttle view event updates to avoid bursts

Test instructions

Unit tests only for now, it could be tested e2e in error collection PR

Checklist

  • Tested locally (playground)
  • Added unit tests for this change.
  • Added e2e/integration tests for this change.

@bcaudan bcaudan force-pushed the bcaudan/main-view-part1 branch from 5cc4f3e to c026c01 Compare February 20, 2026 09:50
@bcaudan bcaudan force-pushed the bcaudan/main-view-part2 branch from 3b35f88 to 64d324c Compare February 20, 2026 09:51
@datadog-datadog-prod-us1-2

datadog-datadog-prod-us1-2 Bot commented Feb 20, 2026

Copy link
Copy Markdown

✅ Code Quality    ✅ Code Vulnerabilities    ✅ Library Vulnerabilities

🎉 All green!

🛠️ No new code quality issues
🛡️ No new code vulnerabilities
📚 No new vulnerable libraries detected

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 0dd3e7c | Docs | Datadog PR Page | Was this helpful? React with 👍/👎 or give us feedback!

@bcaudan bcaudan force-pushed the bcaudan/main-view-part1 branch from c026c01 to 5b1d5a0 Compare February 20, 2026 09:58
@bcaudan bcaudan force-pushed the bcaudan/main-view-part2 branch from 64d324c to acc76ce Compare February 20, 2026 10:03
@bcaudan bcaudan force-pushed the bcaudan/main-view-part1 branch from 5b1d5a0 to 0cf8318 Compare February 20, 2026 10:13
@bcaudan bcaudan force-pushed the bcaudan/main-view-part2 branch 2 times, most recently from 8520226 to 5d2abd6 Compare February 20, 2026 15:41

@cdn34dd cdn34dd left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think this is fine to merge as is, but I'd like to discuss some potential updates next week regarding how we're handling subscriptions. Nothing major, but now that multiple classes can handle multiple subscriptions, tracking data flow can get complicated very fast and I'd like to make it easier to know at a glace what subscribes to what.

@bcaudan bcaudan force-pushed the bcaudan/main-view-part1 branch from 4cbc10f to c3b6312 Compare March 2, 2026 08:55
Base automatically changed from bcaudan/main-view-part1 to main March 2, 2026 08:57
@bcaudan bcaudan force-pushed the bcaudan/main-view-part2 branch from 5d2abd6 to 0dd3e7c Compare March 2, 2026 09:01
@bcaudan bcaudan marked this pull request as ready for review March 2, 2026 09:02
@bcaudan bcaudan merged commit d8c8b00 into main Mar 2, 2026
10 checks passed
@bcaudan bcaudan deleted the bcaudan/main-view-part2 branch March 2, 2026 09:03
@bcaudan bcaudan mentioned this pull request Mar 26, 2026
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.

2 participants