Skip to content

feat: retrieve and show timetagger server version#61

Merged
ardevd merged 2 commits into
mainfrom
timetagger-version-impl
May 30, 2026
Merged

feat: retrieve and show timetagger server version#61
ardevd merged 2 commits into
mainfrom
timetagger-version-impl

Conversation

@ardevd

@ardevd ardevd commented May 30, 2026

Copy link
Copy Markdown
Owner

No description provided.

@ardevd ardevd self-assigned this May 30, 2026
Copilot AI review requested due to automatic review settings May 30, 2026 20:31
@ardevd ardevd added the enhancement New feature or request label May 30, 2026

Copilot AI left a comment

Copy link
Copy Markdown

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 feature to fetch the TimeTagger server version via a new /version API endpoint and display it under the connected server URL in the Settings bottom sheet.

Changes:

  • Adds TimeTaggerVersionResponse data class and getVersion() endpoint on TimeTaggerApiService.
  • Adds a serverVersionText TextView to the settings layout and corresponding strings (English + Norwegian).
  • Fetches the server version in SettingsBottomSheet and toggles visibility on success/failure.

Reviewed changes

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

Show a summary per file
File Description
app/src/main/res/values/strings.xml Adds English "Server Version" string.
app/src/main/res/values-no/strings.xml Adds Norwegian translation.
app/src/main/res/layout/fragment_settings.xml Adds TextView for server version (missing default gone visibility).
app/src/main/java/net/ardevd/tagius/features/settings/ui/SettingsBottomSheet.kt Fetches version and updates UI; swallows exceptions silently.
app/src/main/java/net/ardevd/tagius/core/network/TimeTaggerApiService.kt Adds getVersion() Retrofit endpoint.
app/src/main/java/net/ardevd/tagius/core/data/TimeTaggerVersionResponse.kt New response model.

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

Comment thread app/src/main/res/layout/fragment_settings.xml
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
@ardevd ardevd merged commit 8f9ae60 into main May 30, 2026
2 checks passed
@ardevd ardevd deleted the timetagger-version-impl branch May 30, 2026 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants