Skip to content

Removed versionadded:: 2.1 directives #2548

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
May 4, 2013

Conversation

wouterj
Copy link
Member

@wouterj wouterj commented Apr 24, 2013

Q A
Doc fix? yes
New docs? no
Applies to 2.3+
Fixed tickets -

This PR removes all versionadded:: 2.1 directives from the 2.3 docs, as stated in the Documentation rules.

I doubt if it's usefull to remove them, I think this needs to get done after each time you merge 2.2 into master...

As of Symfony 2.1, you *must* have routes configured for your ``login_path``,
``check_path`` ``logout`` keys. These keys can be route names (as shown
in this example) or URLs that have routes configured for them.

Copy link
Member

Choose a reason for hiding this comment

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

Can we adapt this text to remain, but remove the versionadded? I'm not sure if this is already explained appropriately in this area or if removing this block will mean we're missing an important note.

Copy link
Member

Choose a reason for hiding this comment

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

Actually, I've just checked this area, I think it's fine without this note.

@weaverryan weaverryan merged commit 4346f75 into symfony:master May 4, 2013
@weaverryan
Copy link
Member

Great work on this Wouter - very nice to see the process executed well.

Thanks!

weaverryan added a commit that referenced this pull request May 4, 2013
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.

2 participants