Skip to content

Conversation

@hugovk
Copy link
Member

@hugovk hugovk commented Dec 12, 2025

Description

Copy link
Member

@JacobCoffee JacobCoffee left a comment

Choose a reason for hiding this comment

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

i think this might be better done in fastly?

@JacobCoffee
Copy link
Member

if i understand correctly, what about #2850

@hugovk
Copy link
Member Author

hugovk commented Dec 15, 2025

Yeah, Fastly sounds better, but I'm not too familiar with its config to review it.

@JacobCoffee
Copy link
Member

Ee pointed out a better way, they said /authenticated is hit each time.

class AuthenticatedView(TemplateView):
template_name = "includes/authenticated.html"

so maybe make somehting similar to this or just inherit it even

@hugovk hugovk changed the title Cache different versions of /downloads for Django users Use endpoint for release edit button that's only rendered for staff Dec 17, 2025
@hugovk hugovk force-pushed the edit-this-release-vary branch from 0987164 to 8e1b656 Compare December 17, 2025 17:22
@hugovk
Copy link
Member Author

hugovk commented Dec 17, 2025

Yep, that sounds better, how's this?

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.

2 participants