Skip to content

Conversation

cabutlermit
Copy link
Contributor

@cabutlermit cabutlermit commented Jul 3, 2025

What does this PR do?

  • Add the SearchEngineKeywordsPerformance to the list of installed plugins
  • Add the SearchEngineKeywordsPerformance to the list of active plugins

Helpful background context

The files are in place in the container across all three environments and the ./console plugin:activate command has been run once in all three environments (which preps the database). Now we need to ensure that any container redeploys maintain the active status for this plugin.

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

Check the version of Matomo running in Dev1. The plugin should be installed and listed as "Active" in the list of plugins. I have temporarily added our license key to the dev-matomo instance for this purpose. Once this PR is approved, I will remove the license key from Dev (since we are technically only allowed to use the key on two instances of Matomo).

Includes new or updated dependencies?

NO

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:
The files are in place in the container across all three environments
and the `./console plugin:activate` command has been run once in all
three environments (which preps the database). Now we need to ensure
that any container redeploys maintain the active status for this plugin.

How this addresses that need:
* Add the SearchEngineKeywordsPerformance to the list of installed
plugins
* Add the SearchEngineKeywordsPerformance to the list of active plugins

Side effects of this change:
None.

Relevant ticket(s):
* https://mitlibraries.atlassian.net/browse/INFRA-558
@cabutlermit cabutlermit marked this pull request as ready for review July 3, 2025 19:37
@cabutlermit cabutlermit requested a review from a team as a code owner July 3, 2025 19:37
@cabutlermit cabutlermit requested a review from dhrutibc July 3, 2025 19:37
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.

The plugin is showing up in Dev and is listed as Active

@cabutlermit cabutlermit merged commit ece68a9 into main Jul 3, 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