Skip to content

Conversation

agjohnson
Copy link
Contributor

@agjohnson agjohnson commented Sep 25, 2025

This was the first attempt to fix #12494, then was broken up into multiple stages of work. This is a very minor method addition to the cache tag class, used when project/version are not set on the view instance at the class level. This was an issue in multiple views.

  • Finally adds support for robots.txt cache tagging on a per-version basis
  • Adds missing tests for cache tagging on robots.txt

@agjohnson agjohnson requested a review from a team as a code owner September 25, 2025 00:53
Copy link

read-the-docs-community bot commented Sep 25, 2025

Documentation build overview

📚 dev | 🛠️ Build #29792796 | 📁 Comparing f4b316d against latest (2742aa0)


🔍 Preview build

No files changed.

Copy link

read-the-docs-community bot commented Sep 25, 2025

Documentation build overview

📚 docs | 🛠️ Build #29792797 | 📁 Comparing f4b316d against latest (2742aa0)


🔍 Preview build

Show files changed (2 files in total): 📝 2 modified | ➕ 0 added | ➖ 0 deleted
File Status
build-customization.html 📝 modified
reference/git-integration.html 📝 modified

@ericholscher
Copy link
Member

I think this likely also might have downstream permissions issues on .com. We should double check that these aren't being cached on .com.

@agjohnson
Copy link
Contributor Author

We should double check that these aren't being cached on .com.

This only affects cache tagging for purging, so shouldn't have an effect on caching.

agjohnson added a commit that referenced this pull request Oct 1, 2025
@agjohnson agjohnson changed the title Add cache tags to offline format file downloads Add utility method for explicitly setting cache tags to cache tag class Oct 10, 2025
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.

Proxito: not setting cache tag on offline format files
3 participants