Synced webpages from sitemap to Document model for search. #2284
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sarahboyce
force-pushed
the
sitemap-page-search
branch
2 times, most recently
from
17fef8d to
ca7b952
Compare
sarahboyce
force-pushed
the
sitemap-page-search
branch
from
ca7b952 to
7a42c1e
Compare
sarahboyce
commented
Oct 22, 2025
Comment on lines
+234
to
+239
| if force: | ||
| Document.objects.filter( | ||
| metadata__parents=DocumentationCategory.WEBSITE | ||
| ).delete() |
There was a problem hiding this comment.
In future we could add a last_modified field to Document and check the sitemap last modified and make updates according to this.
But for now, I think having a cron job to build the English dev docs with force (update_docs --force --language=en dev) running every 2 weeks (or even every month) will be good enough to update changes to the webiste pages for search and remove any stale pages. This content doesn't change very often, nor do we remove content often
sarahboyce
force-pushed
the
sitemap-page-search
branch
4 times, most recently
from
8020f62 to
3f4b174
Compare
sarahboyce
force-pushed
the
sitemap-page-search
branch
from
932d8ce to
3bd8c81
Compare
sarahboyce
force-pushed
the
sitemap-page-search
branch
from
3bd8c81 to
1e4380b
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I have included the commits from #2262 so that if someone pulls the branch they get a better picture, only the last 2 commits (3f4b174 and 932d8ce) is new.
Fixes #1499
Note that some changes will be removed once #2136 is merged.
If we are happy with this approach then I may remove
_sync_blog_to_db,_sync_views_to_db,SearchableViewandSEARCHABLE_VIEWSadded in #2136.If we need a "docs only but includes some website pages" filter we can add that in later on, but I can add logic for this if it's a blocker for this change
For testing, recommend running
python manage.py update_docs --force --language=en devBe aware you will get more extensive search results if #2276 is merged