Skip to content

Conversation

cabutlermit
Copy link
Contributor

@cabutlermit cabutlermit commented Sep 16, 2025

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
Copy link
Contributor

@vab vab left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link

@dhrutibc dhrutibc left a comment

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
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