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

Block: cache - remember context #180

Merged
merged 1 commit into from
Jul 11, 2013

Conversation

rmsint
Copy link
Contributor

@rmsint rmsint commented Jul 9, 2013

Q A
Fixed tickets symfony-cmf/block-bundle#67
License MIT

'title': 'Symfony2 CMF news',
'url': 'http://cmf.symfony.com/news.rss',
'maxItems': 2
}) }}
Copy link
Member

Choose a reason for hiding this comment

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

Could you please also add a PHP example:

<?php echo $view['blocks']->render(array(
    'name' => 'rssBlock',
), array(
    'title'    => 'Symfony2 CMF news',
    'url'      => 'http://cmf.symfony.com/news.rss',
    'maxItems' => 2,
)) ?>

(this example only works when the PRs referenced in #179 are merged)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@dbu
Copy link
Member

dbu commented Jul 11, 2013

thomas fixed the sonata doc issue: http://sonata-project.org/bundles/block/master/doc/reference/advanced_usage.html

can you update the link and then we merge this?

@rmsint
Copy link
Contributor Author

rmsint commented Jul 11, 2013

@dbu done and ready to be merged

dbu added a commit that referenced this pull request Jul 11, 2013
@dbu dbu merged commit aa4dc8e into symfony-cmf:master Jul 11, 2013
@dbu
Copy link
Member

dbu commented Jul 11, 2013

thanks!

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