Sync PR for 1.9.0#1852
Merged
subodhr258 merged 11 commits intomainfrom May 7, 2026
Merged
Conversation
Sync main with develop after 1.8.0 release
Fix php warnings
Update: Admin notices as per Marketing
* chore: Rename GoDAM blocks * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
…nent (#1846) * fix: Disable hover option when autoplay is enabled in VideoEdit component * fix: Ensure hover behavior is disabled when autoplay is enabled across components Address the frontend rendering hoverSelect gating --------- Co-authored-by: KMchaudhary <kuldipkumar.chaudhary@rtcamp.com>
🔍 WordPress Plugin Check Report
📊 Report
|
| 📍 Line | 🔖 Check | 💬 Message |
|---|---|---|
0 |
missing_composer_json_file | The "/vendor" directory using composer exists, but "composer.json" file is missing. |
📁 readme.txt (2 warnings)
| 📍 Line | 🔖 Check | 💬 Message |
|---|---|---|
0 |
mismatched_plugin_name | Plugin name "GoDAM - Organize WordPress Media Library & File Manager with Unlimited Folders for Images, Videos & more" is different from the name declared in plugin header "GoDAM". |
0 |
trademarked_term | The plugin name includes a restricted term. Your chosen plugin name - "GoDAM - Organize WordPress Media Library & File Manager with Unlimited Folders for Images, Videos & more" - contains the restricted term "wordpress" which cannot be used at all in your plugin name. |
📁 assets/build/css/main.css (1 warning)
| 📍 Line | 🔖 Check | 💬 Message |
|---|---|---|
0 |
EnqueuedStylesScope | This style is being loaded in all contexts. |
📁 assets/src/libs/analytics.min.js (5 warnings)
| 📍 Line | 🔖 Check | 💬 Message |
|---|---|---|
0 |
EnqueuedScriptsScope | This script is being loaded in all frontend contexts. |
0 |
NonBlockingScripts.NoStrategy | This script on http://localhost:8880 (with handle analytics-library) is loaded in the footer. Consider a defer or async script loading strategy instead. |
0 |
NonBlockingScripts.NoStrategy | This script on http://localhost:8880/2026/05/07/hello-world/ (with handle analytics-library) is loaded in the footer. Consider a defer or async script loading strategy instead. |
0 |
NonBlockingScripts.NoStrategy | This script on http://localhost:8880/sample-page/ (with handle analytics-library) is loaded in the footer. Consider a defer or async script loading strategy instead. |
0 |
NonBlockingScripts.NoStrategy | This script on http://localhost:8880/demo-attachment-post/ (with handle analytics-library) is loaded in the footer. Consider a defer or async script loading strategy instead. |
📁 assets/build/js/main.min.js (5 warnings)
| 📍 Line | 🔖 Check | 💬 Message |
|---|---|---|
0 |
EnqueuedScriptsScope | This script is being loaded in all frontend contexts. |
0 |
NonBlockingScripts.NoStrategy | This script on http://localhost:8880 (with handle rtgodam-script) is loaded in the footer. Consider a defer or async script loading strategy instead. |
0 |
NonBlockingScripts.NoStrategy | This script on http://localhost:8880/2026/05/07/hello-world/ (with handle rtgodam-script) is loaded in the footer. Consider a defer or async script loading strategy instead. |
0 |
NonBlockingScripts.NoStrategy | This script on http://localhost:8880/sample-page/ (with handle rtgodam-script) is loaded in the footer. Consider a defer or async script loading strategy instead. |
0 |
NonBlockingScripts.NoStrategy | This script on http://localhost:8880/demo-attachment-post/ (with handle rtgodam-script) is loaded in the footer. Consider a defer or async script loading strategy instead. |
🤖 Generated by WordPress Plugin Check Action • Learn more about Plugin Check
Contributor
There was a problem hiding this comment.
Pull request overview
This PR synchronizes the plugin release to 1.9.0 and refines frontend/editor behavior and UI copy, with emphasis on preventing conflicting video interactions (autoplay vs hover), improving reliability of admin notices, and reducing “GoDAM”-heavy labeling in block/editor surfaces.
Changes:
- Bumps plugin/package stable tags and versions to 1.9.0.
- Enforces autoplay vs hover-select mutual exclusivity across editor, PHP render, and frontend JS hover manager.
- Updates integration messaging/UI labels (including WooCommerce notice/button copy) and improves a few reliability/accessibility behaviors (instanceId lookup; modal visibility toggling).
Reviewed changes
Copilot reviewed 27 out of 28 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| readme.txt | Updates Stable tag to 1.9.0. |
| README.md | Updates Stable tag to 1.9.0. |
| package.json | Bumps package version to 1.9.0. |
| package-lock.json | Bumps lockfile package version metadata to 1.9.0. |
| godam.php | Bumps plugin header + RTGODAM_VERSION to 1.9.0. |
| inc/templates/godam-player.php | Forces hoverSelect to none when autoplay is enabled; minor PHPCS ignore formatting. |
| assets/src/js/godam-player/managers/hoverManager.js | Skips hover manager initialization when autoplay is enabled. |
| assets/src/js/godam-player/api/player.js | Makes instanceId detection more robust by checking descendants. |
| assets/src/blocks/godam-player/edit.js | Resets hoverSelect to none when autoplay is toggled on; disables hover UI when autoplay is enabled; updates labels. |
| assets/src/blocks/godam-player/block.json | Renames block title to “Video”; fixes textdomain; adds keyword. |
| assets/src/blocks/godam-player/index.js | Updates block icon alt text to be less branded. |
| assets/src/blocks/godam-player/style.scss | Updates comment branding in wrapper styles. |
| assets/src/blocks/godam-player/editor.scss | Updates comment branding in editor styles. |
| assets/src/blocks/godam-pdf/edit.js | Updates placeholder label to “Document”. |
| assets/src/blocks/godam-pdf/block.json | Renames block title to “Document”; adds keyword. |
| assets/src/blocks/godam-audio/edit.js | Updates placeholder label to “Audio”. |
| assets/src/blocks/godam-audio/block.json | Renames block title to “Audio”; adds keyword. |
| assets/src/blocks/godam-gallery-v2/block.json | Renames block title to “Video Gallery”. |
| assets/src/blocks/godam-gallery-v2/edit.js | Updates placeholder label to “Video”. |
| assets/src/blocks/godam-gallery-v2/render.php | Updates frontend placeholder copy to “Video”. |
| assets/src/blocks/godam-gallery-v2/style.scss | Adds visibility toggling for modal/overlay state. |
| pages/video-editor/VideoJSPlayer.js | Allows add-on layer types from localized config; renders add-on icons as <img>. |
| pages/video-editor/style.scss | Adds styling for add-on layer indicator icons rendered as <img>. |
| pages/video-editor/components/SidebarLayers.js | Disables layer selection button when required integration/plugin is inactive. |
| pages/godam/components/tabs/IntegrationsSettings/IntegrationsSettings.jsx | Updates an inline comment. |
| pages/godam/components/tabs/IntegrationsSettings/IntegrationActionButton.jsx | Changes CTA text from “Purchase Now” to “Upgrade Plan”. |
| pages/godam/components/tabs/IntegrationsSettings/integration-tabs.js | Changes WooCommerce badge label from “Pro” to “Beta”. |
| admin/class-rtgodam-transcoder-admin.php | Prevents false “expired key” notices on verification failures; rewrites WooCommerce promo notice messaging. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
KMchaudhary
approved these changes
May 7, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request updates the plugin to version 1.9.0 and introduces several important improvements focused on user experience, accessibility, and reliability, especially around video autoplay and hover interactions. It also includes messaging and branding refinements for a more consistent, less branded UI, and addresses a WooCommerce integration notice. Below are the most significant changes:
Autoplay and Hover Behavior Improvements
VideoEditinedit.js) and the PHP template (godam-player.php). The UI disables the hover option when autoplay is active and updates the help text accordingly. [1] [2] [3] [4]User Interface and Branding Updates
WooCommerce Integration Notice Messaging
Reliability and Error Handling
Styling and Accessibility
visibilityin addition toopacity. [1] [2]Version updates:
These changes collectively enhance the user experience, prevent conflicting behaviors, and modernize the plugin’s interface and messaging.