Core: Change Swagger word in favor of OpenAPI
ismailbaskin
All 13 comments
@ismailbaskin Do you want to make a PR about this ?
Simperfit
on 8 Jun 2017
@Simperfit Should I change all swagger words (except swagger UI) to Open API?
ismailbaskin
on 13 Sep 2017
LGTM... WDYT @api-platform/core-team?
meyerbaptiste
on 13 Sep 2017
Not in class names, it would be a BC break 馃槗.
In UIs it's ok to use Open API. In docs, I think we should use something like Open API (previously known as Swagger) because I'm pretty sure that most people still type "Swagger" when they use the search engine or Google.
dunglas
on 13 Sep 2017
What about api:swagger:export command, it will break BC too. So this issue will be closed until new major version, isn't it? @dunglas
ismailbaskin
on 13 Sep 2017
We can create a new command but keep this one to avoid breaking IMO.
soyuka
on 14 Sep 2017
Wouldn't it work to just use setAliases() @soyuka ?
http://api.symfony.com/3.1/Symfony/Component/Console/Command/Command.html#method_setAliases
antograssiot
on 14 Sep 2017
Yes @antograssiot obviously :p
soyuka
on 14 Sep 2017
@ismailbaskin do you want to do it ?
Simperfit
on 9 Apr 2018
@Simperfit which branch to start on and to which branch this PR should be made ?
mysiar
on 10 Apr 2018
@mysiar master IMHO, since 2.2 does not accept new feature and 2.1/2.0 are bug/security fix only.
Simperfit
on 10 Apr 2018
@Simperfit please correct me if I'm wrong
API documentation standard is OpenAPI but UI is still Swagger, right ?
mysiar
on 10 Apr 2018
dunglas
on 22 Dec 2018
Related issues
soyuka
路
3Comments
mahmoodbazdar
路
3Comments
ghost
路
3Comments
kate-kate
路
3Comments
bendavies
路
3Comments
Most helpful comment
Not in class names, it would be a BC break 馃槗.
In UIs it's ok to use Open API. In docs, I think we should use something like
Open API (previously known as Swagger)because I'm pretty sure that most people still type "Swagger" when they use the search engine or Google.