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

Added a missing dependency to player/library #2318

Merged
merged 1 commit into from
Jul 14, 2017
Merged

Added a missing dependency to player/library #2318

merged 1 commit into from
Jul 14, 2017

Conversation

JulianKniephoff
Copy link
Contributor

Fixes #2317.

Please let me know if there is anything I can do to improve this PR. 🙂

@coveralls
Copy link

Coverage Status

Coverage remained the same at 73.832% when pulling 0ccfc37 on JulianKniephoff:i2317/fix-player-dependency into 5357694 on mediaelement:master.

@rafa8626 rafa8626 merged commit 0ccfc37 into mediaelement:master Jul 14, 2017
@rafa8626
Copy link
Contributor

No everything looks good to me. Merged this PR. Thanks for your work on this and please star https://github.com/mediaelement/mediaelement and https://github.com/mediaelement/mediaelement-plugins to continue supporting this player

@JulianKniephoff
Copy link
Contributor Author

Wow, cool, that was quick. Thanks! 😃

@rafa8626
Copy link
Contributor

You are welcome. Happy to help you guys with this silly but important things that I missed.

@JulianKniephoff JulianKniephoff deleted the i2317/fix-player-dependency branch July 14, 2017 13:03
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