Skip to content

Conversation

@NoumaanAhamed
Copy link
Collaborator

image

Changes suggested to fix the GoDAM Gallery when rtMedia is active.

@rtBot
Copy link
Contributor

rtBot commented Jan 22, 2026

Unable to PHPCS or SVG scan one or more files due to error running PHPCS/SVG scanner:

  • templates/media/godam-integration.php

The error may be temporary. If the error persists, please contact a human (commit-ID: 9aa1f82).

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR prevents the GoDAM integration script from loading on GoDAM video embed pages by adding a conditional check before enqueueing the script.

Changes:

  • Added conditional logic to skip loading rtMedia's GoDAM integration script when on GoDAM video embed pages

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

true
);
// Skip loading rtMedia's GoDAM integration script on Godam video embed pages.
if ( 'video-embed' !== get_query_var( 'godam_page' ) ) {

This comment was marked as off-topic.

Copy link
Member

@mi5t4n mi5t4n left a comment

Choose a reason for hiding this comment

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

LGTM

@NoumaanAhamed NoumaanAhamed merged commit b4dc8b8 into develop Jan 28, 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