Skip to content

DEV-20451: sync system-test fixtures for accurate time-spent metric#133

Draft
caddoo wants to merge 8 commits into
prepare6xfrom
DEV-20451
Draft

DEV-20451: sync system-test fixtures for accurate time-spent metric#133
caddoo wants to merge 8 commits into
prepare6xfrom
DEV-20451

Conversation

@caddoo

@caddoo caddoo commented Jul 9, 2026

Copy link
Copy Markdown

Companion submodule PR for matomo-org/matomo#24788 — updates the checked-in expected fixture(s) so they match the new accurate per-pageview time-on-page output.

Checklist

  • [✔] I have understood, reviewed, and tested all AI outputs before use
  • [✔] All AI instructions respect security, IP, and privacy rules

sgiehl and others added 3 commits July 12, 2026 10:47
Replaces a value re-export/import of a type-only entity so the esbuild-based Vite build can bundle it, and rebuilds the Vue dist files with the new toolchain.
@caddoo
caddoo changed the base branch from 5.x-dev to prepare6x July 14, 2026 21:50
caddoo added 4 commits July 16, 2026 14:20
Companion to matomo-org/matomo DEV-20451. The new accurate archiver
backfills time_spent via visit_last_action_time, so sum_time_spent
and avg_time_on_page shift for last-pageview rows; the interim
nb_hits_with_time_spent metric has been dropped in favor of the
existing nb_hits divisor.
# Conflicts:
#	vue/dist/CustomVariables.umd.js
#	vue/dist/CustomVariables.umd.min.js
…time

The core writer now closes pageviews at the next in-visit action
(closePreviousPageView), so log_page_view_time.time_spent reflects
real wall-clock time rather than the legacy walk-forward sum. Live
visitor log timeSpent updates accordingly (1620s -> 360s for the
ManyVisitsWithGeoIP pageview at t+1h).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants