harmonwood / capacitor-video-player

Capacitor Video Player Plugin
MIT License
112 stars 46 forks source link

Add Capacitor 6 support #154

Closed pormel closed 2 months ago

pormel commented 4 months ago

Please, add support for Capacitor 6 Thanks !

marcjulian commented 4 months ago

Capacitor provides a migration script npx @capacitor/plugin-migration-v5-to-v6@latest updating the files automatically.

https://capacitorjs.com/docs/updating/plugins/6-0#using-capacitorplugin-migration-v5-to-v6

jepiqueau commented 4 months ago

@marcjulian thanks i know it and use it for @capacitor-community/sqlite to provide an alpha version. I am quite busy rewriting the sqlite plugin. So here , sorry you will have to wait.

marcjulian commented 4 months ago

No worries. Thanks for the plugin anyways.

ivancduran commented 4 months ago

Hi @jepiqueau, please before upgrading to version 6, could you please review the pull request I made to add DRM support to the native players. I also tried to contact you, because I need to develop a number of features over the video player and I want to check with you how we can collaborate.

I already sent you and email explaning the situation.

Thanks you very much for your time and for this awesome plugin.

WardPearce commented 4 months ago

This fork exists, can't say if it works or not tho

https://github.com/Jiwoks/capacitor-video-player

nncl commented 2 months ago

That'd be great an update.

Thanks for the awesome work though.

harmonwood commented 2 months ago

Hello, all, I have been granted new maintainer of this project. As an update, Capacitor 6 support will be my main priority, second of course to looking at outstanding PR's. (@ivancduran) Please, bear with me as I get my legs for this project.

eduardoRoth commented 2 months ago

@harmonwood I created a PR for the update to Capacitor v6 (https://github.com/harmonwood/capacitor-video-player/pull/164)

I also have two other PRs for fixes that I found while using this plugin with a client project.

eduardoRoth commented 2 months ago

@harmonwood , let me know if you need something else in the PRs, so I can work on that and we could merge them to master

Thank you! 😄

harmonwood commented 2 months ago

version 6.0.0 has been published to npm