search

InfyOmLabs / laravel-generator

API and Admin Panel CRUD Generator for Laravel.
https://www.infyom.com/open-source
MIT License
3.79k stars 812 forks source link

Swagger package is archived #815

Closed ayman-elmalah closed 4 years ago

ayman-elmalah commented 4 years ago

https://github.com/appointer/swaggervel

this package is archived right now, you can use l5 swagger package instead

https://github.com/DarkaOnLine/L5-Swagger

ajayinfyom commented 4 years ago

we will update Swagger package soon.

aydinbulut commented 4 years ago

I am using L5-Swagger and it works well so far.

epileftro85 commented 4 years ago

Question, the L5-Swagger is working with no additional configuration or do I have to write some Swagger notation because I have ErrorException : Required @OA\Info() not found and I have referred to this but it just no generate the API documentation, just the main controller.

aydinbulut commented 4 years ago

I am using Swagger 2.0 but with a fresh install it uses 3.0 and as far as I remember you have to specify @OA\Info() in one of the PHP classes under app/ directory. I faced this problem when I installed the first time.

Don't forget to re-generate swagger specs.

epileftro85 commented 4 years ago

Sorry if my question is dumb but, would you please give me an example, because InfyOmLabs generates @SWG\Get tag but the above package needs @OA\ tag, it only generates the main controller where I put @SWG\Get.

ajayinfyom commented 4 years ago

@ayman-elmalah @medioseureka @aydinbulut Added support for laravel 7 in Generator Swagger package.

please refer Updated docs: https://labs.infyom.com/laravelgenerator/docs/7.0/generator-options#swagger

also, you can check the video from here: https://www.youtube.com/watch?v=5ILtqB1aR9g

xwiz commented 4 years ago

@ajayinfyom Sorry for reopening, why does the docs prefer swaggervel archived package instead of darkaonline/l5-swagger ? I'm curious because I want to check it out and wondering what might break.

mitulgolakiya commented 4 years ago

@xwiz there is no preference here, it was like, we have support for that package and it was working well. But not I see that for some reason it was archived. so I already forked the repo and now we are using our own forked version. But we can plan to move to some better package in the future.

Or you can do that and put a PR. :)

epileftro85 commented 4 years ago

Hey awsome, thanks for the great job.