Skip to content
This repository was archived by the owner on Sep 16, 2021. It is now read-only.

[WCM] Updated tutorial for new routing auto bundle #507

Merged
merged 8 commits into from
Aug 13, 2014

Conversation

dantleech
Copy link
Member

Q A
Doc fix? no
New docs? yes
Applies to master
Fixed tickets #377

I will also update the RoutingAutoBundle reference in this PR.

@dantleech
Copy link
Member Author

Also i will do a test run of the tutorial before this is can be merged.

Acme\BasicCmsBundle\Document\Post:
url_schema: /post/{date}/{title}
token_providers:
date: [content_date, { method: getDate }
Copy link
Member

Choose a reason for hiding this comment

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

missing ]

@wouterj
Copy link
Member

wouterj commented Jul 1, 2014

@dantleech I think this one is replaced by #511 ?

@dantleech
Copy link
Member Author

No this is the "Creating a CMS ..." tutorial, the #511 is the bundle documentation.

@dantleech
Copy link
Member Author

ok, I finished an entire run-through of this and made the necessary corrections so it works (including the exact composer requirements for this moment in time). We will have to wait until the next stable release before releasing this.

@dbu @lsmith77 do you think we could do a stable release mid to late august?

Also need to address outstanding comments.


Now you will need to include this configuration:
# src/Acme/BasicCmsBundle/Resources/config/cmf_routing_auto.yml
Copy link
Member Author

Choose a reason for hiding this comment

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

@dantleech
Copy link
Member Author

I see that we can change the version on the website, can we merge this into master?

@dantleech
Copy link
Member Author

Updated, it would be great if we could merge this. But I am not sure on the polcy @wouterj @dbu

@wouterj
Copy link
Member

wouterj commented Aug 10, 2014

@dantleech I believe the version switcher is comming to symfony.com. I think the best option is to merge this into master and leave 1.0 with the older formats, so people can still read that.

You can merge this one too when symfony-cmf/routing-auto#9 is merged

@wouterj
Copy link
Member

wouterj commented Aug 11, 2014

there is a sphinx error in this pr

wouterj added a commit that referenced this pull request Aug 13, 2014
…utingauto

[WCM] Updated tutorial for new routing auto bundle
@wouterj wouterj merged commit 070a8c0 into master Aug 13, 2014
@wouterj wouterj deleted the updating_cookbook_for_new_routingauto branch August 13, 2014 20:16
@@ -415,6 +420,8 @@ and add some posts::
}
}

The
Copy link
Member Author

Choose a reason for hiding this comment

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

Hmm

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants