Update to support Laravel 8
hgferreira opened this issue · 15 comments
Hi,
Is it possible to update to support Laravel 8? It should be a quick fix.
Thank you!
same....
same
+1
Up up
+1
I've added a pull request, there are no tests, and I have not tested the package after the updates. #118
+🥰
any updates?
any updates?
Project seems dead. I'd look for a replacement, or fork it and maintain it yourself
any updates?
project seems dead. I'd look for a replacement, or fork it and maintain it yourself
okay thanks
First off, what i did, i wrote an kind email directly to rinvex without a response.
In case it could help anyone too. Here is my solution which has a few downsides and is not perfect at all.
- Modify your composer.json
"autoload": {
"psr-4": {
....
"Rinvex\\": "support/rinvex"
},
},
- Create a directory /support/rinvex or anything you would like
- Move the src Code from rinvex packages into their corresponding directory ( /support/rinvex/Categories|Subscriptions|Support)
- Decide if you want to keep rinvex model validation (I removed it). You might have to do some more work here too to get it working (e.g. you'll need another third party dependency)
- Integrate every dependency that any rinvex package use in your composer.json (like spatie/laravel-sluggable, kalnoy/nestedset, ...)
- Run your tests, if you don't have any, write them.
My win for this solution, i don't have to modify my existing source code and can leave it, as it is. For instance, i'm mostly subclassing package models like seen below
<?php
namespace App\Shared\Models;
class Category extends \Rinvex\Categories\Models\Category
{
}
I don't like this approach but when rinvex comes back i can easily switch back and re-run my tests.
Good luck everyone in surviving this nightmare
+1
Hello, thank you for your patience, this is now done! Check latest release!!
This package now supports Laravel v8, PHP v8, Composer v2, enjoy 🥳
CC @mojtabaahn @schranzer235 @insign @bassem-shoukry @nivv @schranzer235 @alikemalcivelek @masitings @Djuki @Crisis2010 @wimil @hgferreira
thanks @Omranic