feat: show if current version is latest one#1547
Conversation
There was a problem hiding this comment.
Pull request overview
Adds a UI indicator in Settings to show when the installed Desktop UI version matches the latest available version.
Changes:
- Extended
AppVersionInfowith a newisLatestVersionboolean. - Derived
isLatestVersioninuseAppVersion()and displayed a “latest version” label in the Version settings tab.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
renderer/src/common/hooks/use-app-version.ts |
Adds isLatestVersion to the version info returned by the React Query hook. |
renderer/src/common/components/settings/tabs/version-tab.tsx |
Updates the Desktop UI version badge to optionally show a “latest version” label. |
|
Instead of the text, wouldn’t it be better to use a green tag on the side that just says “latest”? |
yeah you're right I don't have a strong opinion. @jtenniswood we can add the a green tag on the right side with latest, wdyt? |
- Fix version comparison to use !isNewVersionAvailable instead of direct string equality to avoid issues with 'v' prefix mismatch - Fix spacing in "(latest version)" label - Add tests for isLatestVersion display behavior - Update test mocks to include isLatestVersion property Co-authored-by: Cursor <cursoragent@cursor.com>
Use !isNewVersionAvailable directly instead of maintaining a separate isLatestVersion field, simplifying the interface and reducing redundancy. Co-authored-by: Cursor <cursoragent@cursor.com>
peppescg
force-pushed
the
show-latest-version-info
branch
from
173afd3 to
db4aa80
Compare
Align version and cli tabs on new rebranding style