Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated to allow H5P items to be embedded onto the course page as an … #232

Open
wants to merge 3 commits into
base: stable
Choose a base branch
from

Conversation

digitalskill
Copy link

allow H5P items to be embedded into course pages as an option.

@digitalskill
Copy link
Author

@thomasmars. Is this an idea the H5P team would consider, adding auto embedding options for h5P items on a course page.

@thomasmars thomasmars requested a review from icc November 28, 2018 08:09
@danmarsden
Copy link
Contributor

FYI - we've had several clients who have wanted to embed H5P content in other locations too (inside book/lesson modules etc.) so one of our devs wrote a small filter that manages this for us.
https://github.com/catalyst/moodle-filter_hvpinsert

Basically you add the h5p activity to the course as usual and then as long as the filter is enabled you basically type:
[[hvp_mod:activity id number]]

where you want the H5P to display anywhere in the course

we haven't dropped this in the plugins db yet, but would be happy for H5P to take it over if you thought it looked useful!

@thomasmars
Copy link
Member

@danmarsden Looks great, I've created an issue ( https://h5ptechnology.atlassian.net/browse/HFP-2476 ) for looking into finding an official use for the filter, should be useful for anyone who embeds H5Ps a lot.

@danmarsden
Copy link
Contributor

cool - it might be able to change to using the embed.php file instead of the custom copy of view.php but I haven't had a chance to look closely at the code. (it's also mainly used on older Moodle or Totara sites so doesn't have the privacy api stuff implemented.)

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.

3 participants