Skip to content

Fix version check to show update banner#1613

Merged
deep1401 merged 2 commits intomainfrom
fix/version-check
Mar 23, 2026
Merged

Fix version check to show update banner#1613
deep1401 merged 2 commits intomainfrom
fix/version-check

Conversation

@deep1401
Copy link
Copy Markdown
Member

No description provided.

@paragon-review
Copy link
Copy Markdown

Paragon Summary

This pull request review identified 2 issues across 2 categories in 1 file. The review analyzed code changes, potential bugs, security vulnerabilities, performance issues, and code quality concerns using automated analysis tools.

This PR fixes a bug in the version check logic that was preventing the update notification banner from displaying when a new version is available, correcting the version comparison in the version service.

Key changes:

  • Fixed version check logic to properly display update banner
  • Modified api/transformerlab/services/version_service.py
  • Resolves issue where update notifications weren't showing

Confidence score: 3/5

  • This PR has moderate risk due to 1 high-priority issue that should be addressed
  • Score reflects significant bugs, performance issues, or architectural concerns
  • Review high-priority findings carefully before merging

1 file reviewed, 2 comments

Severity breakdown: High: 1, Medium: 1


Tip: @paragon-run <instructions> to chat with our agent or push fixes!

Dashboard

@sentry
Copy link
Copy Markdown

sentry bot commented Mar 23, 2026

Codecov Report

❌ Patch coverage is 85.71429% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
api/transformerlab/services/version_service.py 85.71% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@aliasaria aliasaria self-requested a review March 23, 2026 17:20
@deep1401 deep1401 merged commit e9e6912 into main Mar 23, 2026
11 checks passed
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