Skip to content

Prepare for Matomo 6 - #118

Merged
sgiehl merged 12 commits into
6.x-devfrom
prepare6x
Aug 10, 2026
Merged

Prepare for Matomo 6#118
sgiehl merged 12 commits into
6.x-devfrom
prepare6x

Conversation

@sgiehl

@sgiehl sgiehl commented Aug 9, 2026

Copy link
Copy Markdown
Member

Summary

Prepares the Bandwidth plugin for Matomo 6, targeting the new 6.x-dev maintenance branch.

  • plugin.json: version 6.0.0, require.matomo >=6.0.0-b1,<7.0.0-b1
  • CHANGELOG.md: 6.0.0 entry
  • matomo-tests.yml: PHP matrix on the matomo6_min_php / matomo6_max_php aliases, MySQL 8.0 / MariaDB 10.6 database matrix, UI job on Node 24 with an explicit database
  • phpstan.neon: phpVersion 8.1; dropped the vendor/ exclude (the plugin ships no composer.json, so the path never exists and PHPStan 2 fails hard on it) and marked github-action-tests/ optional, since it only exists during a CI run
  • UI screenshots re-synced for headless Chrome (Node 24 / Puppeteer 24), the sparkline feature-flag removal and the report-header search

No code changes were needed: the Matomo 6 breaking-changes sweep found no use of a removed PHP API, HTTP API method, global function, Less variable or JS/Vue API, and the plugin has no Vue sources, no Updates/ files and no JSON controller actions.

Steps to Replicate the Issue

Not applicable — compatibility preparation, no user-facing behaviour change.

Checklist

  • [✔] Tested locally or on demo2/demo3?
  • [NA] New test case added/updated?
  • [NA] Are all newly added texts included via translation?
  • [NA] Are text sanitized properly? (Eg use of v-text v/s v-html for vue)
  • [✔] Version bumped?
  • [✔] I have understood, reviewed, and tested all AI outputs before use
  • [✔] All AI instructions respect security, IP, and privacy rules
  • [NA] Documentation updated?

AI 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 11 commits July 9, 2026 14:43
Bump plugin version to 6.0.0 and require Matomo >=6.0.0-b1,<7.0.0-b1.
* Run plugin tests against MySQL 8.0 and MariaDB 10.6

The workflow inherited the shared action's MySQL 5.7 default, which is below
Matomo's new minimum. Pin the database-backed jobs to the supported floor:
PluginTests now runs a MySQL 8.0 + MariaDB 10.6 matrix, and the UI/JS jobs run
on MySQL 8.0.

* Only upload plugin test artifacts for the MySQL matrix leg

PluginTests runs a MySQL + MariaDB matrix; the upload-artifacts condition
matched both legs and uploaded twice. Restrict the upload to the MySQL leg.
… Puppeteer 24) (#114)

* Update expected UI screenshots for the new headless Chrome (Node 24 / Puppeteer 24)

* Update expected UI screenshots for the new headless Chrome (Node 24 / Puppeteer 24)

* Re-sync expected UI screenshots for the new headless Chrome after rebase

Rendering drift only (identical content, minor line-height/AA shifts) under the
new headless Chrome used by the migrated screenshot-testing harness.

* Run the plugin UI tests on Node 24

Match the Node 24 / Puppeteer 24 screenshot-testing stack; the UI job was still pinned to Node 16.
Teal brand screenshots on 6.x-dev plugin base; synced from matomo-org/matomo run 30405694397.
…ature-flag

updating screenshots for sparkline feature-flag removal
The DataTable search moved from the table footer into the shared ReportHeader
(matomo-org/matomo#24986), which changes every report screenshot.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
…lease]

The literals 8.1/8.5 drift as soon as core's supported range moves; the aliases
resolve centrally in matomo-org/github-action-tests.
PHPStan 2 fails hard on a literal excludePaths entry that matches nothing. The
plugin ships no composer.json, so vendor/ never exists; github-action-tests/ only
exists during a CI run and is marked optional.
@sgiehl
sgiehl requested a review from a team August 9, 2026 14:49
…release]

The report header search collapsed from an expanded input into an icon toggle,
which shifts every report table up by the height of the old input.
Comment thread phpstan.neon
@sgiehl
sgiehl merged commit 1854507 into 6.x-dev Aug 10, 2026
12 checks passed
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.

4 participants