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

Handling more than 50,000 URLs #59

Open
jswilson opened this issue Mar 3, 2020 · 4 comments
Open

Handling more than 50,000 URLs #59

jswilson opened this issue Mar 3, 2020 · 4 comments
Assignees

Comments

@jswilson
Copy link
Contributor

jswilson commented Mar 3, 2020

Hi, just wanted to say thanks for such a great library.

One need we have is to generate a sitemap for a site that has more than 50,000 URLs. The search engines typically only handle a maximum of 50,000 URLs per sitemap file, which means today that we manually create a sitemap index and move the URLs into individual sitemap files, each containing less than 50,000 URLs each.

One option I was considering was adding a feature to python-sitemap that would optionally output a sitemap index and multiple sitemap files if there are more than 50,000 URLs; would that be of interest? Just wanted to make sure that kind of feature would be desired prior to implementing; thanks!

@c4software
Copy link
Owner

First time i seen this point. Seems I will have to chunk the produced sitemap 🤔

@c4software c4software self-assigned this Mar 5, 2020
@c4software c4software pinned this issue Mar 5, 2020
@jswilson
Copy link
Contributor Author

jswilson commented Mar 5, 2020

First time i seen this point. Seems I will have to chunk the produced sitemap 🤔

If you think it would be a good addition, I actually may be able to implement this; I can't quite do it immediately, but could check in with you in a couple weeks about potentially doing it.

@c4software
Copy link
Owner

I confirm. It's a good addition (and is on the spec so 👌). I looked at how to integrate it, but it doesn't seems thats simple to do it. Tell me.

@Garrett-R
Copy link
Contributor

This issue is resolved now that #65 is merged, right?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants