Skip to content

News caching#329

Open
Aaron-Detre wants to merge 7 commits into
developfrom
news-caching
Open

News caching#329
Aaron-Detre wants to merge 7 commits into
developfrom
news-caching

Conversation

@Aaron-Detre

@Aaron-Detre Aaron-Detre commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Changes

  • Split the getNews endpoint into one for the home page and one for dedicated news page.
  • Added optional parameters to the endpoints. They now only return the news items that are needed instead of returning all news items and then filtering those on the client side.
  • Home page news is cached with Redis.

Test

  • Test with News caching WISE-Client#2326
  • If Discourse is not set up and the home page is using admin news:
    • When not signed in, accessing the home page should cache the latest 3 public news items.
    • When signed in, accessing the home page should cache the latest 3 news items of any type.
    • Accessing the dedicated news page should not cache anything.
    • The cache entries should expire each hour or when the list of news items is modified.

@Aaron-Detre Aaron-Detre self-assigned this Jul 7, 2026
@Aaron-Detre Aaron-Detre added the enhancement New feature or request label Jul 7, 2026
@Aaron-Detre Aaron-Detre marked this pull request as ready for review July 8, 2026 02:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant