Skip to content

🐛 [RUM-15689] fix view date to use start time instead of update time#97

Merged
bcaudan merged 1 commit into
mainfrom
bcaudan/fix-view-date
Apr 20, 2026
Merged

🐛 [RUM-15689] fix view date to use start time instead of update time#97
bcaudan merged 1 commit into
mainfrom
bcaudan/fix-view-date

Conversation

@bcaudan

@bcaudan bcaudan commented Apr 17, 2026

Copy link
Copy Markdown
Collaborator

Motivation

View events were missing an explicit date field, causing them to default to the event collection time rather than the view's actual start time. This led to incorrect timestamps on view updates.

Changes

  • Set date to currentView.startTime in emitViewUpdate so every view event carries the correct start timestamp
  • Add test assertions verifying the date on initial and subsequent view updates

Test instructions

Looking at the main view process in RUM Explorer, with this change the relative time should stay at 0s even when the view is updated.

Checklist

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

@bcaudan

bcaudan commented Apr 17, 2026

Copy link
Copy Markdown
Collaborator Author

@codex review

@bcaudan bcaudan marked this pull request as ready for review April 17, 2026 11:50
@bcaudan bcaudan requested a review from a team as a code owner April 17, 2026 11:50
@bcaudan bcaudan merged commit ce38d8d into main Apr 20, 2026
14 checks passed
@bcaudan bcaudan deleted the bcaudan/fix-view-date branch April 20, 2026 15:26
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