Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.3k
remove the security.listener.factory DIC tag#2585
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
remove the security.listener.factory DIC tag#2585
Uh oh!
There was an error while loading.Please reload this page.
Conversation
xabbuh commentedMay 2, 2013
| Q | A |
|---|---|
| Doc fix? | yes |
| New docs? | no |
| Applies to | 2.1+ |
| Fixed tickets | #2347 |
wouterj commentedMay 2, 2013
we maybe can add a versionadded directive stating that this is removed.@weaverryan wdyt? |
…-dic-tagremove the security.listener.factory DIC tag
weaverryan commentedMay 4, 2013
Thanks for the good PR! @wouterj - If 2.1 were the current version, then I'd agree with you that we should add a note. We'd then remove that note on the 2.3 branch later on (like we recently did with other 2.1 notes). In this case, I think we can avoid doing the extra work of having this note on just the 2.1 and 2.2 branch since this is all already in the past. So, we'll save a little trouble :). Cheers! |