Skip to content

Support for Symfony 3 #28

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

Merged
merged 1 commit into from
Jan 3, 2016
Merged

Support for Symfony 3 #28

merged 1 commit into from
Jan 3, 2016

Conversation

peshi
Copy link
Contributor

@peshi peshi commented Dec 14, 2015

SecurityContext was removed in 3.0 - the purpose of this PR is to provide a BC layer for older versions of Symfony.

@peshi
Copy link
Contributor Author

peshi commented Dec 14, 2015

Once FOSOAuthServerBundle is updated for Symfony 3, allow_failure for 3.0 should then be removed from travis.yml.

@thomask
Copy link

thomask commented Dec 29, 2015

Awesome, was just about to start on this PR myself. Hope this'll get merged soon.

@jaytaph
Copy link
Owner

jaytaph commented Dec 29, 2015

I will try and see if I can merge this today/tomorrow, time permitting.. I probably need some SF3 support in the near future myself as well :)

@jaytaph
Copy link
Owner

jaytaph commented Jan 3, 2016

@peshi Thanks for your work! Much appreciated. We'll probably want to support the 2.3 version for a while, and I'm really hoping FOSOAuth will be upgraded soon too (FriendsOfSymfony/FOSOAuthServerBundle#374)

jaytaph added a commit that referenced this pull request Jan 3, 2016
@jaytaph jaytaph merged commit d5caf1f into jaytaph:master Jan 3, 2016
@peshi
Copy link
Contributor Author

peshi commented Jan 3, 2016

@jaytaph No worries, thank you for merging 👍

@peshi peshi deleted the symfony3 branch January 3, 2016 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants