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

Generate sitemaps from CLI #99

Merged
merged 2 commits into from Jul 31, 2019
Merged

Conversation

stefandoorn
Copy link
Owner

Basic implementation to generate the sitemaps (+ index) from CLI. Plenty of improvements needed, e.g.:

  • Splitting into service (+ specs)
  • Renderer should return file names generated, which the index builder should take in account
  • Separating builder from provider collections (both for builder + index builder)

And plenty of other improvements.

I would like to merge this one first, so in consecutive pull requests enhancements can be made. Tests pass again, so we are save to develop on top of this. This pull request is already rather big.

@stefandoorn stefandoorn force-pushed the feature/generate-cli-using-providers branch from d10a4eb to 0f957d5 Compare July 29, 2019 16:21
@stefandoorn stefandoorn force-pushed the feature/generate-cli-using-providers branch from 0f957d5 to c893655 Compare July 29, 2019 16:25
src/Command/GenerateSitemapCommand.php Outdated Show resolved Hide resolved
src/Controller/SitemapController.php Show resolved Hide resolved
src/Filesystem/Writer.php Show resolved Hide resolved
src/Resources/config/config.yaml Show resolved Hide resolved
src/Resources/config/services/sitemap.xml Show resolved Hide resolved
src/Resources/config/services/sitemap.xml Show resolved Hide resolved
@stefandoorn stefandoorn force-pushed the feature/generate-cli-using-providers branch from e4e2c97 to 582ecfc Compare July 31, 2019 18:24
@stefandoorn stefandoorn merged commit e838679 into master Jul 31, 2019
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.

None yet

2 participants