Skip to content

feat: Add version indicators to lineage graph nodes#6187

Merged
franciscojavierarceo merged 4 commits intomasterfrom
update-lineage
Mar 27, 2026
Merged

feat: Add version indicators to lineage graph nodes#6187
franciscojavierarceo merged 4 commits intomasterfrom
update-lineage

Conversation

@franciscojavierarceo
Copy link
Copy Markdown
Member

@franciscojavierarceo franciscojavierarceo commented Mar 27, 2026

Summary

  • Adds version badges (v2, v3, etc.) to Feature View nodes in the lineage graph when they have currentVersionNumber > 0
  • Hovering the badge shows a tooltip with the version number, total versions from history, and latest version description
  • Adds a "Version Changed" entry to the lineage legend
  • Fixes a pre-existing eslint warning (missing permissions dependency in useEffect)

Screenshot

Notice the "Version Changed" in the Legend section and the bottom right corner of each Feature View.

Screenshot 2026-03-27 at 9 15 48 AM

Test plan

  • 7 new tests in RegistryVisualization.test.tsx covering:
    • Version badge renders for regular, on-demand, and stream feature views with currentVersionNumber > 0
    • No badge when version is 0 or absent
    • Version history data from featureViewVersionHistory is passed through
    • Legend shows "Version Changed" entry
  • All 11 existing UI tests continue to pass
  • Manual verification via npm start — lineage page shows badges on versioned nodes

🤖 Generated with Claude Code


Open with Devin

Display version badges on Feature View nodes in the lineage graph when
they have been versioned (currentVersionNumber > 0). Badges show a
tooltip with version details and total version count from the registry's
version history. Also adds a "Version Changed" entry to the legend and
fixes a missing `permissions` dependency in useEffect.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@franciscojavierarceo franciscojavierarceo requested a review from a team as a code owner March 27, 2026 13:04
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

franciscojavierarceo and others added 3 commits March 27, 2026 09:20
Version 1 is the initial version, so a badge is only meaningful when the
feature view has actually changed (version 2+). Also fixes test name to
match the updated threshold and adds a test for the v1 case.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Apply prettier formatting to fix CI formatting checks.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@franciscojavierarceo franciscojavierarceo merged commit 73805d3 into master Mar 27, 2026
35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants