Skip to content
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

Add example of YAML configuration #120

Merged
merged 2 commits into from
Jun 14, 2016

Conversation

bocharsky-bw
Copy link
Contributor

No description provided.

@yann-eugone
Copy link
Member

I think that we also could add support for XML and more complicated value than true or false.
I will push as comment some examples

@yann-eugone
Copy link
Member

yml_homepage:
    path:     /
    defaults: { _controller: AppBundle:Default:index }
    options:
        sitemap: true
yml_homepage:
    path:     /
    defaults: { _controller: AppBundle:Default:index }
    options:
        sitemap:
            section: yml
<route id="xml_homepage" path="/">
    <default key="_controller">AppBundle:Default:index</default>
    <option key="sitemap">true</option>
</route>
<route id="xml_homepage" path="/">
    <default key="_controller">AppBundle:Default:index</default>
    <option key="sitemap">
        {"section":"xml"}
    </option>
</route>

@bocharsky-bw
Copy link
Contributor Author

@yann-eugone Hey, thanks for this examples! I've added simple YAML and XML examples of route configuration here in this PR. Maybe expanded configuration examples better move to the separate docs page instead of README? What do you think?

@yann-eugone
Copy link
Member

@bocharsky-bw yes, I agree with this. i will do it later, can you open an issue ?

@bocharsky-bw
Copy link
Contributor Author

@yann-eugone I did!

@yann-eugone yann-eugone merged commit 74366eb into prestaconcept:master Jun 14, 2016
@bocharsky-bw bocharsky-bw deleted the patch-1 branch June 14, 2016 17:40
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.

None yet

2 participants