8000 [Book] consistent and complete config examples by xabbuh · Pull Request #4114 · symfony/symfony-docs · GitHub
[go: up one dir, main page]

Skip to content

[Book] consistent and complete config examples #4114

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
Aug 16, 2014

Conversation

xabbuh
Copy link
Member
@xabbuh xabbuh commented Aug 10, 2014
Q A
Doc fix? yes
New docs? no
Applies to all
Fixed tickets

Make configuration examples in the book consistent throughout the different configuration formats. Also fixing several syntax issues.

<access-control>
<rule path="^/cart/checkout" role="IS_AUTHENTICATED_ANONYMOUSLY"
<rule path="^/cart/checkout"
role="IS_AUTHENTICATED_ANONYMOUSLY"
requires_channel="https" />
Copy link
Member Author

Choose a reason for hiding this comment

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

Can someone confirm that requires_channel is the right attribute (and note requires-channel)?

Copy link
Member

Choose a reason for hiding this comment

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

it should be requires-channel (both work, but according to the Symfony XML standards, dashes should be used)

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks.

Make configuration examples in the book consistent throughout the
different configuration formats. Also fixing several syntax issues.
@weaverryan
Copy link
Member

WOW, another HUGE PR. I don't know how you get such a huge volume of changes :). Thanks as always @xabbuh!

@weaverryan weaverryan merged commit 96fb64f into symfony:2.3 Aug 16, 2014
weaverryan added a commit that referenced this pull request Aug 16, 2014
This PR was merged into the 2.3 branch.

Discussion
----------

[Book] consistent and complete config examples

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | all
| Fixed tickets |

Make configuration examples in the book consistent throughout the different configuration formats. Also fixing several syntax issues.

Commits
-------

96fb64f [Book] consistent and complete config examples
@xabbuh xabbuh deleted the consistent-book-code-examples branch August 16, 2014 17:18
weaverryan added a commit that referenced this pull request Aug 16, 2014
This PR was merged into the 2.3 branch.

Discussion
----------

[Components] consistent & complete config examples

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies t
8EFD
o    | all
| Fixed tickets |

Make configuration examples in the components section consistent throughout the different configuration formats. Also fixing several syntax issues.

This is mainly the same that I did in #4114 for the book.

Commits
-------

d08da50 [Components] consistent & complete config examples
weaverryan added a commit that referenced this pull request Aug 25, 2014
This PR was merged into the 2.3 branch.

Discussion
----------

[Reference] consistent & complete config examples

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | all
| Fixed tickets |

Make configuration examples in the reference section consistent throughout the different configuration formats. Also fixing several syntax issues.

This follow #4114 and #4129.

Commits
-------

01dc25c [Reference] consistent & complete config examples
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.

3 participants
0