Skip to content

Conversation

@ro0NL
Copy link
Contributor

@ro0NL ro0NL commented Aug 18, 2016

Q A
Branch? 2.8
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? yes
Tests pass? yes
Fixed tickets #19664, #19661, #19652
License MIT
Doc PR -

By the grace of @chalasr

ping @fabpot

@chalasr
Copy link
Member

chalasr commented Aug 18, 2016

Manually checked out and it fixed it 👍

@ro0NL
Copy link
Contributor Author

ro0NL commented Aug 18, 2016

I dont know wtf is going on.. (github? fabpot?).. but #19662 was removed.. now it's back. Anyway, for reference :-)

@chalasr
Copy link
Member

chalasr commented Aug 18, 2016

I thought the owner has been banned and the related issues/PRs too...

@ro0NL
Copy link
Contributor Author

ro0NL commented Aug 18, 2016

Me too, but in case of no censor; 👍

@nicolas-grekas
Copy link
Member

👍 for 2.8 since this is not really a new deprecation but rather a bc-break fix.
Code is exactly the same on MainConfiguration, we just forgot to add it here also by the time.

@nicolas-grekas nicolas-grekas changed the title [SecurityBundle] Deprecate form_login.intention [SecurityBundle] Add missing deprecation notice for form_login.intention Aug 19, 2016
@fabpot
Copy link
Member

fabpot commented Aug 19, 2016

Thank you @ro0NL.

fabpot added a commit that referenced this pull request Aug 19, 2016
…ogin.intention (ro0NL)

This PR was squashed before being merged into the 2.8 branch (closes #19667).

Discussion
----------

[SecurityBundle] Add missing deprecation notice for form_login.intention

| Q             | A
| ------------- | ---
| Branch?       |  2.8
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | yes
| Tests pass?   | yes
| Fixed tickets | #19664, #19661, #19652
| License       | MIT
| Doc PR        | -
By the grace of @chalasr

ping @fabpot

Commits
-------

2a7bbd1 [SecurityBundle] Add missing deprecation notice for form_login.intention
@fabpot fabpot closed this Aug 19, 2016
@ro0NL ro0NL deleted the issue/19664 branch August 23, 2016 11:00
This was referenced Sep 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants