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

[WCM][Console] add documentation for the new method added for table dividers #3269

Closed
wants to merge 1 commit into from

Conversation

cordoval
Copy link
Contributor

@cordoval cordoval commented Dec 3, 2013

Q A
Doc fix? no
New docs? yes ( symfony/symfony#9680 )
Applies to 2.5+
Fixed tickets -

@wouterj
Copy link
Member

wouterj commented Dec 3, 2013

fyi, I changed the PR template to use our template: http://symfony.com/doc/current/contributing/documentation/overview.html#pull-request-format

@@ -23,6 +23,7 @@ set headers, rows and render::
array('960-425-059-0', 'The Lord of the Rings', 'J. R. R. Tolkien'),
array('80-902734-1-6', 'And Then There Were None', 'Agatha Christie'),
))
->setDividersAt(array(2))
Copy link
Member

Choose a reason for hiding this comment

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

I don't know if I like this, because they don't really no what this exactly does

@xabbuh
Copy link
Member

xabbuh commented Dec 8, 2013

Can you add [WCM] to the PR title?

@wouterj
Copy link
Member

wouterj commented Dec 8, 2013

@xabbuh done :)

@xabbuh
Copy link
Member

xabbuh commented Dec 8, 2013

👍

@fabpot
Copy link
Member

fabpot commented Mar 3, 2014

Closing as symfony/symfony#10365 implements this feature in another way. See #3627 for some documentation about it.

@fabpot fabpot closed this Mar 3, 2014
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.

4 participants