Develop to Main Sync for v1.5.0#1523
Merged
KMchaudhary merged 40 commits intomainfrom Jan 20, 2026
Merged
Conversation
Sync PR from Main to Dev after 1.4.9
* fix: Robust background scroll fix for iPhone 13 Pro and others * fix: Check for dvh support before using it
* feat: Add filter to skip automatic transcoding on upload * feat: Update Transcoding text * fix: Sanitize array of ids Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Fix: Empty CTA screen on incomplete details filled. #1162 * Removed default image from CTA image container * Fix fatal error if CTA image is removed from media library * display upload option if physical image is removed from media library
* feat: responsive analytics page * fix: review automated review comments --------- Co-authored-by: Subodh Rajpopat <subodh.rajpopat@rtcamp.com>
…ive (#1416) * Fixed Image CTA (Portrait) Cut Off in Safari Instead of Being Responsive * Updated text wrap property for title andd description in image cta * fix: Improve Vertical Image CTA scroll behaviour --------- Co-authored-by: Subodh Rajpopat <subodh.rajpopat@rtcamp.com>
* fix: Hotspot positions in portrait and fullscreen * fix: Backward Compatibility with legacy hotspots * fix: Clamping based on resize and drag both * fix: Hotspot apearing over the controls * fix: Apply copilot feedbacks * fix: Use consistent hotspot size for new hotspots * fix: Add new hotspot layer with initial hotspot of the correct size
#1457) * Initial plan * Fix poll confirmation detection and preserve button icon Co-authored-by: subodhr258 <58080782+subodhr258@users.noreply.github.com> * Refactor arrow icon creation and optimize text node search Co-authored-by: subodhr258 <58080782+subodhr258@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: subodhr258 <58080782+subodhr258@users.noreply.github.com>
Co-authored-by: KMchaudhary <kuldipkumar.chaudhary@rtcamp.com>
* Implement godam-embed video page, and integrate on godam-gallery block * Implement the responsive width height for godam embed video * Implement iframe on gallery-block and remove the unwanted API calls and make code clean * Implement loading animation and improve the gallary item scroll behaviour * Fix the iframe URL issue on dynamic load * Implement iframe postMessages to trigger previous/next video load on gallery block * Conditionaly add style if godam block is rendered threw iframe * Improve the godam engagement style for iframe * Provide engagements query parameter to show engagements along with video * Add lofig to only show engagement options if engagement enabled and video is transcoded * Fix the engagment options not working issues * Add loading spinner until engagment popup is not opened * Fix the gallery block larger screen height issue * Make godam-gallery block style responsive * Introduce video close button * Fix the engagements query parameter issue for dynamically loaded gallery videos * Fix the iframe fullscreen for iOS devices * Show single CPT page on gallery-video if open_to_new_tab attributes is enabled * Resolve the copilot feedback * Hide the godam gallery modal's close button on fullscreenmode * Fix the transcoded status casing issue * Fix the godam video title issue for non-transcoded videos * Resolve PR feedback * Restirct the video scroll on forms, cta, and polls layers * Stop scrolling of video on fullscreen --------- Co-authored-by: KMchaudhary <kuldipkumar.chaudhary@rtcamp.com>
* Fix godam block apsectRation reflection issue on block editor * Resolved copilot comments * Improve the code comment * Fix the godam/video height and aspect ratio related issue on block editor --------- Co-authored-by: KMchaudhary <kuldipkumar.chaudhary@rtcamp.com>
Co-authored-by: KMchaudhary <kuldipkumar.chaudhary@rtcamp.com>
* Fix the fullscreen font-size issue for all forms, CTA, and polls layers * Address copilot comment --------- Co-authored-by: KMchaudhary <kuldipkumar.chaudhary@rtcamp.com>
* feat: Add PostHog support in GoDAM * feat: Add PostHog settings to allow user to disable tracking * fix: Update settings help text Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * feat: Update package.json Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * fix: Update PostHog setting in Help page, Opt-in methods * feat: Add Admin Notice for usage tracking * fix: Console error on Help page * fix: Update notice message * fix: Make tracking anonymous * fix: PostHog initialization * fix: Enqueue JS for admin notice --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Fixed Image CTA (Portrait) Cut Off in Safari Instead of Being Responsive * Updated text wrap property for title andd description in image cta * fix: Player controls for minimal skin * fix: Play button vertical alignment * fix: Skip buttons alignment in Default skin * fix: Video loading animation play button on mobile screen * fix: Share button not clickable * fix: Remove outline from video player elements * fix: Make CTA image layer (landscape) responsive * fix: No left padding in control bar for Pills skin * fix: Captions on Safari * fix: Image CTA double container in video editor * fix: Make CTA image layer responsive on inline video * fix: Remove Share button from iOS fullscreen * fix: Seeker overlapping over captions menu * fix: Default Skin play button slightly to the left * fix: CTA title should have ellipsis after exceeding 2 lines * fix: Hover effects for menu in Bubble Skin * fix: Time start and time duration in bubble skin * fix: Time flickering issue in Bubble skin * fix: Hide share button in all fullscreens * fix: Horizontally align play button in classic skin * fix: Improve Vertical Image CTA scroll behaviour * fix: CTA scroll behaviour in portrait mode * fix: Make CTA consistent in video editor and frontend * fix: Remove extra gap in container * fix: Remove unnecessary else if condition * fix: Minial skin play button for desktop * fix: Increase CTA description text size * fix: Keyboard play/pause indicator icon * fix: Undo centering of play button for minimal and pills skins * fix: Center the big play button for both mobile and desktop * fix: Keyboard vjs-play indicator --------- Co-authored-by: opurockey <opu.rockey@gmail.com>
Co-authored-by: KMchaudhary <kuldipkumar.chaudhary@rtcamp.com>
Co-authored-by: KMchaudhary <kuldipkumar.chaudhary@rtcamp.com>
* fix: Add support for Row and Stack in block editor * fix: Blue hover issue on GoDAM Blocks icons * fix: Add alt text to icon images
* Implement virtual media support for audio, image, and PDF blocks * Fix the core/image attempt block recovery issue Create block extension script that will help us extend the core blocks editor logic * Implement image versions callback endpoint * Add subtype=pdf check for pdf attachments * Only show the oEmbed field on medias popup if its video attachment * Add transcoding URL, and virtual status for audio, pdf, and image attachments This will show the correct transcoding status for default and virtual media imported from GoDAM central * Fix the virtual file name, add extension after filename * Fix the icon preview based on file type for virtual attachments * Add support for virtual pdf media thumbnail support * Completed the image subsizes generation, and rendering on frontend for image attachments Remove debug statements * Handle image version generation if all requested versions are already available * Fix the fullsize image's srcset domain mismatch issue * Remove unwanted changes * Resolve copilot feedback * Resolve the second iteration of copilot issues * Fix video thumbnail regresion issue for virtual media * Resolve the PR feedback * Resolve breaking merge conflicts * Update the use of URL validation function --------- Co-authored-by: KMchaudhary <kuldipkumar.chaudhary@rtcamp.com>
* Started storing the transcoded image url in attachment postmeta * Resolve copilot comments --------- Co-authored-by: KMchaudhary <kuldipkumar.chaudhary@rtcamp.com>
… Ninja Forms #1182 (#1502) * Fix: GoDAM Recorder links are not working as expected for WPForms and Ninja Forms #1182 * Update inc/classes/ninja-forms/class-ninja-forms-integration.php Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update inc/classes/ninja-forms/class-ninja-forms-integration.php Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Moved Ninja submission list script to standalone file --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Implement the loading animation on godam_video block Show loading animation for first time until videojs player is ready * Resolve copilot comments * fix the style loading issue for some sites --------- Co-authored-by: KMchaudhary <kuldipkumar.chaudhary@rtcamp.com>
* Show the oEmbed URL for godam tab video items * fix the audio, pdf thumbnail url if thumbnail url not recevied from godam-central --------- Co-authored-by: KMchaudhary <kuldipkumar.chaudhary@rtcamp.com>
* Fix buttons overflow issue * Fix underline issue * Fix graph issue * Adress github bot comments * fix: Analytics page header and text --------- Co-authored-by: Subodh Rajpopat <subodh.rajpopat@rtcamp.com>
* fix: Volume slider for Bubble skin * fix: Text CTA overflow and Quill Styles * fix: Limit height of VideoJS menus * fix: Close button width on mobile fullscreen * fix: Font Weight * fix: Time Control in bubble skin for Engagements modal * fix: Add space in comment Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * fix: Add missing semicolon Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * fix: Add horizontal padding to Text CTA * fix: Wrong share button in Bubble Skin --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Fix the image transcoding retranscoding requests Restrict the retranscoding request for images when virtual media is created * Address typos --------- Co-authored-by: KMchaudhary <kuldipkumar.chaudhary@rtcamp.com>
* Fix the url missmatch of virtual image source set * Resolve copilot comments --------- Co-authored-by: KMchaudhary <kuldipkumar.chaudhary@rtcamp.com>
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Co-authored-by: KMchaudhary <kuldipkumar.chaudhary@rtcamp.com>
rtBot
reviewed
Jan 19, 2026
Contributor
rtBot
left a comment
rtBot
left a comment
There was a problem hiding this comment.
Code analysis identified issues
action-phpcs-code-review has identified potential problems in this pull request during automated scanning. We recommend reviewing the issues noted and that they are resolved.
phpcs scanning turned up:
Powered by rtCamp's GitHub Actions Library
|
|
||
| // Check if asset file exists (generated by @wordpress/scripts). | ||
| if ( file_exists( $block_extensions_asset_file ) ) { | ||
| $block_extensions_asset = include $block_extensions_asset_file; |
Contributor
There was a problem hiding this comment.
$block_extensions_asset_file). Probably needs manual inspection (WordPressVIPMinimum.Files.IncludingFile.UsingVariable).
🔍 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. |
📁 inc/classes/media-library/class-media-folder-utils.php (1 warning)
| 📍 Line | 🔖 Check | 💬 Message |
|---|---|---|
91 |
PluginCheck.Security.DirectDB.UnescapedDBParameter | Unescaped parameter $where_clause used in $wpdb->get_var($wpdb->prepare(\n\t\t\t\t"\n\t\t\tSELECT COUNT(DISTINCT p.ID)\n\t\t\tFROM {$wpdb->posts} p\n\t\t\tINNER JOIN {$wpdb->term_relationships} tr ON p.ID = tr.object_id\n\t\t\tINNER JOIN {$wpdb->term_taxonomy} tt ON tr.term_taxonomy_id = tt.term_taxonomy_id\n\t\t\tWHERE {$where_clause}\n\t\t",\n\t\t\t\t$query_params\n\t\t\t))\n$where_clause assigned unsafely at line 82:\n $where_clause .= ' AND p.post_mime_type LIKE %s'\n$query_params[] used without escaping.\n$mime_type used without escaping. |
📁 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 (6 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/?p=1 (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/?page_id=2 (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/?attachment_id=6 (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/?godam-video=demo-godam-video-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 (6 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/?p=1 (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/?page_id=2 (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/?attachment_id=6 (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/?godam-video=demo-godam-video-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
…1530) * Fix the gallery block console errors, occure due to rtmedia plugin * Fix github advance security issue: DOM text reinterpreted as HTML --------- Co-authored-by: KMchaudhary <kuldipkumar.chaudhary@rtcamp.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request syncs v1.5.0 from develop to main, introducing PostHog analytics integration, GoDAM virtual media support for multiple media types, Safari/iOS video player improvements, mobile-responsive design updates, and numerous bug fixes.
Changes:
- Integrated PostHog analytics with user-configurable tracking settings
- Implemented virtual media support for Image, Audio, and PDF imports from GoDAM Central
- Enhanced video player compatibility for Safari/iOS, including fullscreen handling and quality menu support
- Improved mobile responsiveness across Dashboard, Analytics, Gallery, and video player components
Reviewed changes
Copilot reviewed 107 out of 108 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| webpack.config.js | Added new build entries for video embed, ninja forms submissions, and block extensions |
| pages/utils/posthog.js | New PostHog initialization utility with configuration validation |
| pages/*/index.js | Wrapped React apps with PostHogProvider for analytics tracking |
| pages/video-editor/ | Fixed aspect ratio format, improved hotspot positioning with percentage-based calculations |
| pages/analytics/ | Made analytics page mobile-responsive with improved chart scaling |
| pages/godam-gallery.js | Refactored to use iframe-based video embed with scroll navigation |
| inc/classes/rest-api/ | Added image resize callback endpoint, transcoding status checks, virtual media handling |
| inc/helpers/custom-functions.php | Added embed page content generation, improved error handling |
| assets/src/js/godam-player/ | Enhanced Safari compatibility, iOS fullscreen fixes, improved hotspot layer positioning |
| assets/src/css/ | Mobile-responsive improvements for all player skins and engagement modal |
| inc/templates/video-embed.php | New template for iframe-embedded videos with engagement support |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| </div> | ||
| <div className="flex justify-center flex-wrap gap-8 py-6 max-w-[1260px] mx-auto px-4"> | ||
| { content.map( ( section, index ) => ( | ||
| { content && content.map( ( section, index ) => ( |
There was a problem hiding this comment.
This guard always evaluates to true.
Collaborator
Author
There was a problem hiding this comment.
Extra validation will not break functionality
…est (#1538) Co-authored-by: KMchaudhary <kuldipkumar.chaudhary@rtcamp.com>
Co-authored-by: KMchaudhary <kuldipkumar.chaudhary@rtcamp.com>
* Replace the doc commetn @SInCE version string n.e.x.t with 1.5.0 * Add changelogs, update versions, and add new contributors profiles * Generate new POT file for release v1.5.0 --------- Co-authored-by: KMchaudhary <kuldipkumar.chaudhary@rtcamp.com>
* Add new contributors * Add kishu7270 --------- Co-authored-by: KMchaudhary <kuldipkumar.chaudhary@rtcamp.com>
kishan-gondaliya-7270
approved these changes
Jan 20, 2026
KMchaudhary
added a commit
that referenced
this pull request
Jan 20, 2026
This reverts commit f29261e.
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.
GoDAM Plugin v1.5.0
Changelog