Skip to content

Conversation

@learning-equality-bot
Copy link

This PR was automatically created by the Update H5P JS library Github Action.
Updates from target repository commit: 52634986bf139fc3e9d9e917b7b03caac2577b9a
h5p/h5p-php-library@1afd89a...5263498

Target repo commit: 52634986bf139fc3e9d9e917b7b03caac2577b9a
Copy link
Member

@rtibbles rtibbles left a comment

Choose a reason for hiding this comment

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

I don't see anything problematic in the H5P diff - it's just some minor styling changes, but maybe it will mess with our H5P rendering. Just needs QA confirmation of our test H5P resources to make sure nothing is awry in terms of styling and visibility of elements.

@rtibbles rtibbles self-assigned this Oct 16, 2025
@pcenov
Copy link
Member

pcenov commented Oct 17, 2025

@rtibbles - manual QA checks out, no H5P regressions observed.

Copy link
Member

@rtibbles rtibbles left a comment

Choose a reason for hiding this comment

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

Manual QA checks out.

@rtibbles rtibbles merged commit 05aeac4 into develop Oct 18, 2025
58 checks passed
@rtibbles rtibbles deleted the h5p_update branch October 18, 2025 04:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants