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

PeerTube Videos support #1544

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Conversation

younes200
Copy link

@younes200 younes200 commented Nov 17, 2022

Hi,
We use react-player in our open source project celluloid, a small contribution to add support for PeerTube videos.

Its follow the same structure from other players component.

Peertube is a great alternative to centralized video platforms (https://en.wikipedia.org/wiki/PeerTube)

@younes200
Copy link
Author

younes200 commented Feb 29, 2024

Hi @cookpete,
The PR is ready to merge ;)
Tested and working on production, example: https://celluloid.huma-num.fr/project/4878b2b9-706b-46b7-af9b-b2a0eb23eae7

@younes200
Copy link
Author

Can you please merge this in some way?

@luwes
Copy link
Collaborator

luwes commented Mar 15, 2024

hey @younes200, really appreciate all your work on this!

For now it doesn't look like many users are asking / voting for this feature so it makes less sense to merge this in the core.

There are methods for adding custom players https://github.com/cookpete/react-player#adding-custom-players which would fit this use case better I think.

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.

2 participants