Skip to content

Installation of Custom Reports Plugin - #30

Merged
cabutlermit merged 1 commit into
mainfrom
infra-576
Sep 17, 2025
Merged

cabutlermit merged 1 commit into
mainfrom
infra-576

Conversation

@cabutlermit

@cabutlermit cabutlermit commented Sep 16, 2025 •

Copy link
Copy Markdown
Contributor

What does this PR do?

UXWS requested the installation of the Custom Reports premium plugin. The prep work has already been completed (loading the files into the container). Now it's time to move on to enabling the plugin and ensuring that the container definition is up-to-date by updating the config.ini.php file to match the changes that appeared in the running container.

How can a reviewer manually see the effects of these changes?

The Custom Reports plugin has been activated in the dev-matomo instance. The reviewer can log in to dev-matomo and play around with the custom reports to ensure that it is accessible. Further testing will take place after this PR has merged to main and we deploy the updates in stage-matomo.

Includes new or updated dependencies?

YES!

What are the relevant tickets?

Developer

  • [n/a] All new ENV is documented in README (or there is none)
  • Stakeholder approval has been confirmed (or is not needed)

Code Reviewer

  • The commit message is clear and follows our guidelines
    (not just this pull request message)
  • There are appropriate tests covering any new functionality
  • The documentation has been updated or is unnecessary
  • The changes have been verified
  • New dependencies are appropriate or there were no changes

Why these changes are being introduced:
UXWS Requested the installation of the Custom Reports premium plugin.

The prep work has already been completed (loading the files into the
container). Now it's time to move on to enabling the plugin and
ensuring that the container definition is up-to-date.

How this addresses that need:
* Update the config.ini.php file to match the changes that appeared
in the running container

Side effects of this change:
None.

Relevant ticket(s):
* https://mitlibraries.atlassian.net/browse/INFRA-576
@cabutlermit
cabutlermit marked this pull request as ready for review September 16, 2025 20:27
@cabutlermit
cabutlermit requested a review from a team as a code owner September 16, 2025 20:27

@vab vab left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

@dhrutibc dhrutibc left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Yes, this looks good in Dev. I can't finish creating a report because there is no data to pull from, but the functionality of creating a report works as expected

@cabutlermit
cabutlermit merged commit 245b840 into main Sep 17, 2025
2 checks passed
@cabutlermit
cabutlermit deleted the infra-576 branch August 3, 2026 16:05
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.

3 participants