-
Notifications
You must be signed in to change notification settings - Fork 142
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
Add Laravel 7 support #323
Comments
What needs to be changed? Lavacharts is framework agnostic, so technically it already can be used? Are the Laravel specific features not working? (I haven't been in PHP land a while, the last Laravel release I was excited about included auto-discovery for packages) |
Hello @kevinkhill It creates this error: Do you have any suggestions to workaround it? |
I fiiiiiiiiiinally downloaded L7 and linked my local copy of Lavacharts... now we blowing off the cobwebs.... do some dusting.... No issue so far with L7 and Lavacharts 3.1, just made a quick PieChart We'll track the progress of Composer 2.0 testing with this issue |
Hi! Any updates on this issue please? Laravel is now up to version 8, and Composer 2.0 has been out for some time. |
I have been out of the PHP land for a bit, so L5.x was my last working
site. Do you know what it would take? Easy upgrade? I don't have time to
learn L7 and upgrade Lavacharts. My life is just too busy at the moment. I
would gladly accept pull requests if you can make the upgrade work.
|
@kevinkhill you might want to check out #322 |
Could you try the latest tag? I have other users reporting no more issues. |
@kevinkhill 3.1.14 seems to have fixed this. Thanks for putting together this package, it's very much appreciated! |
Hi Kevin,
Could you update composer.json to support Laravel 7 as well?
Thanks
The text was updated successfully, but these errors were encountered: