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

No sitemap no robots #2578

Closed
wants to merge 2 commits into from
Closed

Conversation

Bromind
Copy link
Contributor

@Bromind Bromind commented Jul 19, 2024

Addresses feature request #2248

this simply adds two fields in config.toml, one field no_sitemap, which disable the generation of sitemap.xml if set, and one field no_robots, which disable the generation of no_robots.txt if set.

IMPORTANT: Please do not create a Pull Request adding a new feature without discussing it first.

The place to discuss new features is the forum: https://zola.discourse.group/
If you want to add a new feature, please open a thread there first in the feature requests section.

This feature is discussed in #2248 "We could make it a config.toml option." (Keats, July 2023) -> This is exactly what this PR implements

Sanity check:

  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Code changes

(Delete or ignore this section for documentation changes)

  • Are you doing the PR on the next branch?

If the change is a new feature or adding to/changing an existing one:

  • Have you created/updated the relevant documentation page(s)?

@Bromind
Copy link
Contributor Author

Bromind commented Jul 19, 2024

messed up with my PR, I'll create a new clean one

@Bromind Bromind deleted the no_sitemap_no_robots branch July 19, 2024 10:38
@Bromind Bromind restored the no_sitemap_no_robots branch July 19, 2024 10:39
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.

1 participant