Laravel8 + MySQL 5.7
Nizari opened this issue · 11 comments
@grimzy Thank you for your hard work. Any idea when this package wil support laravel8 + mysql 5.7? Or is this already possible.
I've also just has a problem not being able to install the 2.x branch on Laravel 8 as well. Would this be complex to support?
This would be a very welcome addition. Anything I can do to help make this possible?
Same here. Cannot upgrade MySQL for many reasons and, right now, we cannot upgrade to Laravel 8 just because of this package (all our other dependencies now support it). We’ll do our own fork of version 2.2.3 for the time being, but it would be great to keep the ability to use the ‘official’ repository.
@Nizari Sure. So far, I did only two things:
- I allowed Laravel 8 in the list of Composer dependencies, like this.
- Then I changed the visibility of a method that the package was overwriting from Laravel, like that.
So far, this works for us. But note that we only use a small part of the features that are provided by the package, so there may be additional changes that could be required to make it work in your case.
Also, if you need a fork, please make your own: I cannot guarantee that we will keep our own online, as we may look for an alternative package at some point.
Hope this helps :)
@Nizari Sure. So far, I did only two things:
- I allowed Laravel 8 in the list of Composer dependencies, like this.
- Then I changed the visibility of a method that the package was overwriting from Laravel, like that.
So far, this works for us. But note that we only use a small part of the features that are provided by the package, so there may be additional changes that could be required to make it work in your case.
Also, if you need a fork, please make your own: I cannot guarantee that we will keep our own online, as we may look for an alternative package at some point.
Hope this helps :)
Thank you very much @miclf. We’ll check it out!
Please add support for Laravel 8 (Liege-bike@f953d02)
+1
i created a PR for this: #155
if you could merge & tag i would hugely appreciate it, as i really need this for a client