Skip to content

feat(Home Page): News caching#329

Merged
hirokiterashima merged 8 commits into
developfrom
news-caching
Jul 9, 2026
Merged

feat(Home Page): News caching#329
hirokiterashima merged 8 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 feat(Home Page): 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

@hirokiterashima hirokiterashima left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@hirokiterashima hirokiterashima merged commit 79d817e into develop Jul 9, 2026
2 checks passed
@hirokiterashima hirokiterashima deleted the news-caching branch July 9, 2026 17:44
@hirokiterashima

Copy link
Copy Markdown
Member

🎉 This PR is included in version 1.21.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@Aaron-Detre Aaron-Detre changed the title News caching feat(Home Page): News caching Jul 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants