Skip to content

Conversation

@NoumaanAhamed
Copy link
Collaborator

  • FIXED
    • Fixed End to End playwright testing scripts for media validation.
    • Fixed minor dashboard frontend bug caused due to build process.

@rtBot
Copy link
Contributor

rtBot commented Nov 20, 2025

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

  • app/assets/admin/js/admin.min.js
  • app/assets/admin/js/rtmedia-admin.min.js
  • app/assets/js/rtmedia.min.js
  • index.php

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

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 updates rtMedia to version 4.7.6, addressing two key issues:

  • Fixed End-to-End Playwright testing scripts for media validation with improved cleanup and more robust assertions
  • Fixed minor dashboard frontend bug through CSS build process improvements

Key Changes

  • Enhanced E2E test reliability with proper cleanup mechanisms using afterEach hooks
  • Improved test assertions by breaking down exact string matching into flexible individual checks
  • Fixed variable naming inconsistency in music type test (changed from video to music)
  • Corrected spelling errors in test descriptions ("disbale" → "disabling")
  • Regenerated minified CSS/JS assets with vendor prefix cleanup

Reviewed Changes

Copilot reviewed 9 out of 24 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tests/wp-e2e-playwright/specs/types.spec.js Added cleanup mechanism with afterEach hook, improved test assertions, fixed typos and variable naming
index.php Updated plugin version to 4.7.6
readme.txt Added changelog entry for 4.7.6 and updated stable tag
README.md Added changelog entry for 4.7.6
languages/buddpress-media.pot Updated POT file version and creation date
app/assets/css/* Regenerated CSS minified files with vendor prefix cleanup
app/assets/js/rtmedia.min.js Regenerated minified JS file
app/assets/admin/* Regenerated admin CSS/JS minified files

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

@nayemDevs nayemDevs merged commit 60d674e into master Nov 20, 2025
14 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.

5 participants