Skip to content
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

Allow "friendsofsymfony/rest-bundle:^3.0" #1175

Merged
merged 1 commit into from
Aug 19, 2020

Conversation

wbloszyk
Copy link
Member

@wbloszyk wbloszyk commented Jul 24, 2020

Subject

Allow "friendsofsymfony/rest-bundle:^3.0".

I am targeting this branch, because these changes respect BC.

Changelog

### Added
- support for "friendsofsymfony/rest-bundle:^3.0"
### Change
- support for deprecated "rest" routing type in favor for xml

@@ -13,7 +13,8 @@

namespace Sonata\PageBundle\Controller\Api;

use FOS\RestBundle\Controller\Annotations\View;
use FOS\RestBundle\Controller\Annotations as Rest;
use FOS\RestBundle\View\View as FOSRestView;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why we need this alias?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is much easier to read this be people with less knowlage about REST and nelmio/api-doc.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should not create aliases when they are not required.

src/Resources/config/routing/api.xml Outdated Show resolved Hide resolved
@wbloszyk wbloszyk requested a review from phansys August 1, 2020 13:09
@wbloszyk wbloszyk marked this pull request as ready for review August 1, 2020 13:10
@wbloszyk wbloszyk force-pushed the support_rest_3 branch 5 times, most recently from 9fcbf57 to 0795b30 Compare August 18, 2020 10:57
@wbloszyk
Copy link
Member Author

@phansys @jordisala1991 Can you check it and ping contributors? It is RTM.

@VincentLanglet VincentLanglet requested a review from a team August 18, 2020 11:46
composer.json Show resolved Hide resolved
@wbloszyk wbloszyk requested a review from core23 August 18, 2020 13:30
@VincentLanglet VincentLanglet merged commit 5152095 into sonata-project:3.x Aug 19, 2020
@VincentLanglet
Copy link
Member

Thanks @wbloszyk

@wbloszyk wbloszyk deleted the support_rest_3 branch August 19, 2020 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants