-
Notifications
You must be signed in to change notification settings - Fork 88
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
Laravel 6.0 support #59
Comments
Just opened a PR on #60 |
Actually, just bumping the version is not enough. Vimeo/laravel use the package vimeo/vimeo-api that use ankitpokhrel/tus-php locked to version 0.1.0. The problem is, ankitpokhrel/tus-php 0.1.0 require Carbon 1 and not Carbon 2 that is the main requirement of laravel 6 |
not supported yet. |
@asterism612 after the last PR it should work. Can you please define what's wrong about it? |
I still get this
|
I've changed it on Line 19 in 76c9cc7
|
I can see that, also it's been merged, hasn't it? However, has it been tagged? Because I am using
in |
Oh I see. @muglug can you please tag and release latest commits on master as 5.4.1 ? |
@hassanazimi can you try using dev-master on your composer for now? |
can't do that! composer stops working when updating! My whole app upgrade stuck because of this! |
Maybe the problem is only on 'how' it was tagged for packagist. and works fine for me until this moment. |
Well for me when I use
and I run |
I suggest deleting your composer.lock and try composer install while keeping vimeo/laravel on dev-master as it is. |
@muglug can you please tag and release latest commits on master as 5.4.1 ? Still no support for Laravel 6. |
Same problem: Is there any issue? Thanks! |
@hassanazimi hassanazimi are You found a solution of this problem? |
My fix for problem: |
@uapixart Thanks your solution worked for me. |
Any chance you can make this package compatible for Laravel 6 and publish in packagist as soon as you can
The text was updated successfully, but these errors were encountered: