-
Notifications
You must be signed in to change notification settings - Fork 156
[WCM] Updated tutorial for new routing auto bundle #507
Conversation
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 } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing ]
@dantleech I think this one is replaced by #511 ? |
No this is the "Creating a CMS ..." tutorial, the #511 is the bundle documentation. |
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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This depends on symfony-cmf/routing-auto-bundle#114
I see that we can change the version on the website, can we merge this into master? |
@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 |
there is a sphinx error in this pr |
…utingauto [WCM] Updated tutorial for new routing auto bundle
@@ -415,6 +420,8 @@ and add some posts:: | |||
} | |||
} | |||
|
|||
The |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm
I will also update the RoutingAutoBundle reference in this PR.