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

fix the RoutingAutoBundle dependency in the getting started guide #485

Closed
lsmith77 opened this issue May 12, 2014 · 5 comments
Closed

fix the RoutingAutoBundle dependency in the getting started guide #485

lsmith77 opened this issue May 12, 2014 · 5 comments
Assignees
Milestone

Comments

@lsmith77
Copy link
Member

https://github.com/symfony-cmf/symfony-cmf-docs/pull/484/files#r12533534

/cc @dantleech

@lsmith77
Copy link
Member Author

@wouterj as we cannot fix this without a stable RoutingAutoBundle release, imho we should for now remove this tutorial from the docs

@dbu
Copy link
Member

dbu commented May 26, 2014

or can we relax the requirements on RoutingAutoBundle to support cmf 1.1 so that the tutorial works again? it will still be not nice that its using the non-stable routing auto bundle, but just removing the tutorial would also be sad. instead of doing a different tutorial, i would rather we bring the routing auto bundle to a released version asap.

@dantleech
Copy link
Member

I agree with @lsmith77 about removing the tutorial. The new version of RoutingAuto is quite a big departure in terms of configuration.

The new version is pretty much ready, I am just waiting for doctrine/orm#1022 to be (hopefully) merged into the ORM, after that we can add the endFlush event to the ODM and the RoutingAutoBundle will be feature complete.

Although I guess the new event would be phpcr-odm 1.2 - as it is only needed for creating RedirectRoutes maybe we could just not support that in 1.1

@dbu
Copy link
Member

dbu commented May 26, 2014

now it seems an even worse idea of mine to have moved the phpcr-odm tutorial into this tutorial - as discussed in #495. if we hide the complete tutorial, we also hide the phpcr-odm setup section which would be fine.

apart from that, we could hide it and link to https://github.com/dbu/conference-tutorial/pulls until we can reactivate the tutorial.

@dantleech
Copy link
Member

The new RoutingAuto is now merged in master and the docs are updated. Closing this.

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

No branches or pull requests

3 participants