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

Add sitemap.xml generation #404

Merged
merged 33 commits into from
May 18, 2022
Merged

Add sitemap.xml generation #404

merged 33 commits into from
May 18, 2022

Conversation

caendesilva
Copy link
Member

@caendesilva caendesilva commented May 18, 2022

About

When a site_url is set, (and the feature is not disabled in the config), a sitemap.xml will automatically be generated during the static site building process.

Features

  • Adds sitemap.xml generation

Changes

  • ext-simplexml has been added as a requirement to composer.json
  • The order of page builders have been changed, so that Blade and Markdown pages are compiled first.
    This allows you to browse the most common parts of the site (like the index page) while the other
    pages are being generated.
  • Merge "Site URL Configuration" options. Please republish your configs.

caendesilva and others added 27 commits May 18, 2022 16:26
It should not be possible for a file to disappear in the microsecond since the collection was generated. If a stat fails it is probably because the input parameter was wrong, which would be an error and not something that should have a graceful fallback.
Adds a helper that handles shared code to make it easier to manage and maintain.

Does not change compiled output.
@caendesilva caendesilva linked an issue May 18, 2022 that may be closed by this pull request
2 tasks
@caendesilva caendesilva marked this pull request as draft May 18, 2022 16:30
@caendesilva caendesilva marked this pull request as ready for review May 18, 2022 16:54
@caendesilva caendesilva merged commit 3b7c798 into master May 18, 2022
@caendesilva caendesilva deleted the add-sitemap branch May 18, 2022 16:57
@caendesilva caendesilva mentioned this pull request May 18, 2022
2 tasks
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.

Add sitemap and/or RSS feed
2 participants