Skip to content

New feature label#5366

Open
hanars wants to merge 2 commits intodevfrom
new-feature-label
Open

New feature label#5366
hanars wants to merge 2 commits intodevfrom
new-feature-label

Conversation

@hanars
Copy link
Copy Markdown
Collaborator

@hanars hanars commented Mar 27, 2026

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a “New” indicator next to the Feature Updates header link by surfacing the most recent Feature Update publish date from the backend (cached in Redis) into the React app’s initial meta JSON.

Changes:

  • Fetch latest Feature Updates Atom feed entry publish date server-side and cache it in Redis.
  • Expose meta.lastFeatureUpdate to the UI via a new Redux selector.
  • Render a “New” Label in the header when the latest update is within the last month; update/extend react app page tests accordingly.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
ui/shared/components/page/Header.jsx Renders “New” label next to Feature Updates link based on lastFeatureUpdate.
ui/redux/selectors.js Adds selector to read meta.lastFeatureUpdate.
seqr/views/react_app.py Adds lastFeatureUpdate to initial page JSON; fetches/caches latest publish date via Redis + GitHub feed.
seqr/views/apis/feature_updates_api.py Refactors feed fetch into fetch_feature_updates() for reuse.
seqr/views/react_app_tests.py Adds mocking for feed/Redis and asserts lastFeatureUpdate behavior in initial JSON.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@hanars hanars marked this pull request as ready for review March 27, 2026 15:56
@hanars hanars requested a review from bpblanken March 27, 2026 15:56
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