-
Notifications
You must be signed in to change notification settings - Fork 156
Conversation
8ce5200
to
f832daf
Compare
@ElectricMaxxx may i ask you to review this one as well? |
~~~~~~~~~~~ | ||
|
||
.. versionadded:: 1.3 | ||
Support for sitemaps was added in version 1.3 of the SeoBundle. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"was introduced in SeoBundle 1.3."
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep our last version was 1.1.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Except the version thing, this documentation looks very good and explains the complecity of the configuration in a very short way. Great. |
updated, thanks for the reviews
|
@dbu this is your friendly reminder service: please be aware of the changes made by symfony-cmf/seo-bundle#243 thanks. |
updated for default sitemaps |
@ElectricMaxxx ok if we merge this and then document the features we are now cleaning up in separate PRs? would be more manageable imho. |
👍 would be nice if this is merged, an i can use the reference on my
|
documenting symfony-cmf/seo-bundle#225 assuming that symfony-cmf/seo-bundle#239 is also merged.
fix #681