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

[SitemapBundle] Make use of a sitemapindex and sub sitemaps #330

Merged
merged 1 commit into from
Jun 2, 2015
Merged

[SitemapBundle] Make use of a sitemapindex and sub sitemaps #330

merged 1 commit into from
Jun 2, 2015

Conversation

denbatte
Copy link
Contributor

Q A
Bug fix? no
New feature? yes
BC breaks? yes
Deprecations? no
Fixed tickets #4

WHAT
This feature generates one central sitemap.xml file at your document root to define a sub sitemap for each language available.
You can now have up to 50000 urls for each language you define.
As a bonus, you only have to register the sitemap index (/sitemap.xml) at the search engines to register all available languages.

HOW
In order to upgrade, you need a new route in your app/config/routes.yml.

KunstmaanSitemapBundle_sitemapIndex:
    resource: "@KunstmaanSitemapBundle/Controller/SitemapController.php"
    type:     annotation

TODO

  • add a note in UPGRADE-3.2.md file

@krispypen
Copy link
Contributor

@roderik 👍

@roderik
Copy link
Contributor

roderik commented Apr 15, 2015

@krispypen we need to defer this to 3.2, it features a bc break because it adds a new route. Currently i'm using master as 3.1.1-dev. Let's keep this one open for a while.

@roderik
Copy link
Contributor

roderik commented Jun 2, 2015

@denbatte please create an UPGRADE-3.2.md doc and add this info

roderik pushed a commit that referenced this pull request Jun 2, 2015
[SitemapBundle] Make use of a sitemapindex and sub sitemaps
@roderik roderik merged commit 50829c5 into Kunstmaan:master Jun 2, 2015
@denbatte denbatte deleted the feature/sitemapUpdate branch June 2, 2015 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants