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

Add a note about RouteDefaultsValidator #694

Merged
merged 1 commit into from
Aug 24, 2015

Conversation

EmmanuelVella
Copy link
Contributor

Refs #667

@dbu I'm don't know what you exactly want to document here, so I just added a small note about it.

@dbu
Copy link
Member

dbu commented Aug 3, 2015

thanks!
i thought we could explain the validator for people that write their own route model. the validator itself is not specific to phpcr-odm. and then this note would just mention that its using the validator. the validator doc could go into dynamic_customize.rst because its only relevant when doing your own stuff.

@EmmanuelVella EmmanuelVella force-pushed the route-defaults-validator branch 2 times, most recently from 84f432d to 9722272 Compare August 10, 2015 12:50
.. note::

The ``RouteDefaultsValidator`` validates the route defaults parameters.
For more information, see :ref:`<_bundle-routing-route-defaults-validator>`.
Copy link
Member

Choose a reason for hiding this comment

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

you need to write the ref without the leading _ for sphinx to find the target

@EmmanuelVella EmmanuelVella force-pushed the route-defaults-validator branch 3 times, most recently from 36e7baf to 4062f83 Compare August 24, 2015 10:08
@EmmanuelVella
Copy link
Contributor Author

@dbu Thanks ! Is it ok or do you want me to add something else ?

dbu added a commit that referenced this pull request Aug 24, 2015
@dbu dbu merged commit 70b206a into symfony-cmf:master Aug 24, 2015
@lsmith77 lsmith77 removed the wip/poc label Aug 24, 2015
@dbu
Copy link
Member

dbu commented Aug 24, 2015

thanks!

@EmmanuelVella EmmanuelVella deleted the route-defaults-validator branch August 24, 2015 11:06
@dbu dbu mentioned this pull request Sep 17, 2015
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