-
-
Notifications
You must be signed in to change notification settings - Fork 16
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
Total refactoring #21
Comments
hi @antonkomarev, what will it breaks if you remove the requirement of laravel 5? |
In theory L6 and L7 support could be added without removing L5 support. |
this in you composer I'm trying to install your branch add-latest-laravel-versions-support |
Yes. I haven't added latest Laravel versions because this package was archived for a long time. You could send a PR for the new Laravel versions support. |
@antonkomarev does the laravel7 branch work with laravel 7, or does the package still need to be updated to laravel7 |
This package is not working with latest laravel versions. I'm not using it on production anymore. It requires some refactoring. |
Is this package still needed "Total Refactoring" to be an ongoing concern? |
@jny986 I've removed abandoned message. Package was updated at the end of the 2020, but I forgot to remove this notice. |
I don't mind helping with the development as well.> Is this package still needed "Total Refactoring" to be an ongoing concern?
|
@aaly00 not really, if you are fine with current implementation - we could continue development without changing its architecture and create separate package with the new one requirements when needed. It will be awesome if you will write how are you using this package and what you want to improve. |
Supports Laravel 8/9? |
@andrekutianski Laravel 8 is supported. You may provide a PR with Laravel 9 support if you want. |
This package requires total refactoring before adding new Laravel versions support and new features.
The text was updated successfully, but these errors were encountered: