sentry icon indicating copy to clipboard operation
sentry copied to clipboard

feat(Replay): Add URLs Visited to EAP Replay Query

Open cliffordxing opened this issue 1 month ago • 1 comments

Currently when executing the EAP replay query, we see the following error:

image

This is because the frontend expects the urls visited to be passed in the data, but the EAP query previously did not query this data. Thus, this PR adds a new query to post process the URLs visited to the replay data. Specifically, the query groups by the URL column, selecting the URL itself along with the earliest time stamp. We then sort this afterwards to ensure correctness of ordering.

Unit tests are also added to assert correctness of URL retrieval, ordering, and deduping.

cliffordxing avatar Dec 08 '25 19:12 cliffordxing

Codecov Report

:x: Patch coverage is 83.33333% with 3 lines in your changes missing coverage. Please review. :white_check_mark: All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...y/replays/endpoints/organization_replay_details.py 83.33% 3 Missing :warning:
Additional details and impacted files
@@             Coverage Diff             @@
##           master   #104538      +/-   ##
===========================================
+ Coverage   80.51%    80.53%   +0.01%     
===========================================
  Files        9352      9353       +1     
  Lines      400281    400874     +593     
  Branches    25700     25700              
===========================================
+ Hits       322274    322829     +555     
- Misses      77539     77577      +38     
  Partials      468       468              

codecov[bot] avatar Dec 08 '25 19:12 codecov[bot]