search

codeforcauseorg / edu-client

Product focuses on 100% education as well as upskilling developing countries and rural areas.
https://educlient.codeforcause.org/
MIT License
130 stars 149 forks source link

Video Player Consistency #149

Closed rizwan2000rm closed 2 years ago

rizwan2000rm commented 3 years ago

Describe the bug

There are currently two different video player packages. video-react and react-player. I think we only need one npm package which can be used for every purpose. React Player seems to be a better option as it is still being maintained.

Additional context

If we agree on either package, we should remove the other package and use same player everywhere.

rizwan2000rm commented 3 years ago

@KeenWarrior @Abhishek-kumar09 ping!

Abhishek-kumar09 commented 3 years ago

Do you want to work on it?

rizwan2000rm commented 3 years ago

@rizwan2000rm Yeah sure. I will work on it.