-
-
Notifications
You must be signed in to change notification settings - Fork 99
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
Wrong support of Laravel 6 #105
Comments
Hmm, good point, apologies for that. @tomirons thoughts on if there's a backwards compatible way to support L6 here or if we should remove that support and try to document what the last version that worked with L6 was? I assume it was 1.x before our 2.x updates? |
I suggest removing support for L6. |
I use this library on a L10 project, it was amazing how easy is to have an entire postman collection. After that I try it on another project, that is biggest, but over L6 :( I try another postman generators, but this is the most active and with better configurations. |
It depends on if there would be a heavy lift in order to support L6 at this point. If not, I would be more than willing to look at a PR that adds support. |
Laravel 6 is suppored in
composer.json
but in\AndreasElia\PostmanGenerator\Processors\RouteProcessor::processRoute()
method use\Illuminate\Support\Str::of()
that has been added after Laravel 6 (I think in v7 or v8).So when I run
php artisan export:postman
there is no routes in json and have many log errors like "Failed to process route: ...".If I write
dd($e->getMessage());
oncatch
part, when I run command get'sMethod Illuminate\Support\Str::of does not exist.
The text was updated successfully, but these errors were encountered: