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

Add SeoBundle twig extension documentation #705

Merged
merged 1 commit into from
Oct 1, 2015

Conversation

benglass
Copy link
Member

Q A
Doc fix? no
New docs? yes (symfony-cmf/seo-bundle#263)
Applies to SeoBundle master/1.2


This bundle provides a twig function ``cmf_seo_update_seo_page``
which lets you set the object which should be used to populate
the seo page. For details on how this works, read :doc:`introduction`.
Copy link
Member

Choose a reason for hiding this comment

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

s/this/populating the SEO page/

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed

@lsmith77 lsmith77 added review and removed wip/poc labels Sep 23, 2015
The twig extension was added in SeoBundle 1.2.

This bundle provides a twig function ``cmf_seo_update_metadata``
which lets you set the object which should be used to populate
Copy link
Member

Choose a reason for hiding this comment

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

is it changing the object or does it just extract information from that object, in addition to what it already extracted from previous objects?

@dbu
Copy link
Member

dbu commented Sep 30, 2015

please also add a note in the documentation about extracting the metadata that mentions the twig functions in case documents are looped over in twig or otherwise not yet available in the controller.

The twig extension was added in SeoBundle 1.2.

This bundle provides a twig function ``cmf_seo_update_metadata``
which lets you populate the seo page from an object.
Copy link
Member Author

Choose a reason for hiding this comment

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

@dbu I reworded this per your comment.

@benglass benglass force-pushed the add_twig_extension_update_seo_page_docs branch from 952b224 to a7e8ad8 Compare September 30, 2015 17:02
{% extends 'base.html.twig' %}

{% block metadata %}
{{ cmf_seo_update_metadata(post) }}
Copy link
Member Author

Choose a reason for hiding this comment

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

@dbu Removed the do, you caught me trying to be clever.

dbu added a commit that referenced this pull request Oct 1, 2015
…o_page_docs

Add SeoBundle twig extension documentation
@dbu dbu merged commit cb68ed6 into master Oct 1, 2015
@dbu dbu deleted the add_twig_extension_update_seo_page_docs branch October 1, 2015 15:26
@dbu
Copy link
Member

dbu commented Oct 1, 2015

thanks!

@lsmith77 lsmith77 removed the review label Oct 1, 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