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

Automatically generate sitemap.xml after building HTMLs #89

Merged
merged 2 commits into from
Aug 31, 2022

Conversation

refeed
Copy link
Member

@refeed refeed commented Jul 18, 2022

Closes #70

@rht
Copy link
Contributor

rht commented Jul 18, 2022

Do you have a sample GH Pages instance to test this?

@refeed
Copy link
Member Author

refeed commented Jul 19, 2022

Not yet, gonna create it now

@refeed refeed force-pushed the sitemap_generator branch 4 times, most recently from 0be47df to 732defa Compare July 19, 2022 06:38
@refeed
Copy link
Member Author

refeed commented Jul 19, 2022

@rht
Copy link
Contributor

rht commented Jul 19, 2022

The sitemap.xml and the .gz looks good. The remaining thing is to add fullstop at the end of your commit messages.

@rht
Copy link
Contributor

rht commented Jul 19, 2022

After this PR, it would be a good idea to test the sitemap.xml content when the number of URLS are > 15k.

@refeed refeed force-pushed the sitemap_generator branch from 732defa to 073355f Compare July 19, 2022 09:29
@rht
Copy link
Contributor

rht commented Jul 19, 2022

LGTM

@refeed refeed marked this pull request as ready for review July 19, 2022 10:00
@lukasstevens
Copy link

Is there any progress on this? This feature would be very useful.

@timabbott timabbott merged commit 980675c into zulip:master Aug 31, 2022
@timabbott
Copy link
Member

Merged, thanks @refeed!

Perhaps you can add documentation of this feature in a follow-up PR?

@refeed
Copy link
Member Author

refeed commented Aug 31, 2022

Sure, will do this weekend!

@refeed
Copy link
Member Author

refeed commented Sep 10, 2022

Just opened a PR for the docs at #92

eric-wieser added a commit to eric-wieser/zulip-archive that referenced this pull request Jan 15, 2023
This was a typo introduced in zulip#89
@@ -32,8 +32,7 @@ python3 get-pip.py
pip install virtualenv
virtualenv -p python3 .
source bin/activate
pip3 install zulip==0.6.3
pip3 install pyyaml==5.2
pip3 install install -r requirements.txt
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

install is here twice: #96

timabbott pushed a commit that referenced this pull request Jan 19, 2023
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.

Automatically generate sitemap.xml after the archive is built.
6 participants