Skip to content

Conversation

@graycreate
Copy link
Member

Summary

  • Fixed bug where Settings -> About showed hardcoded version "版本1.0.0"
  • Now dynamically reads actual version and build number from Bundle
  • Displays in format "版本v1.1.0(3)" as requested

Changes

  • Updated SettingsPage.swift to read version from CFBundleShortVersionString and build from CFBundleVersion
  • Removed hardcoded version string

Test plan

  • Build project successfully
  • Verify version displays correctly in format "版本vX.X.X(Y)"
  • Confirmed values match Info.plist settings

🤖 Generated with Claude Code

- Changed hardcoded version "版本1.0.0" to dynamically read from Bundle
- Now displays version in format "版本v1.1.0(3)" with actual version and build number
- Version info is pulled from CFBundleShortVersionString and CFBundleVersion

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings September 14, 2025 15:12
Copy link

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

This PR fixes a hardcoded version display issue in the Settings About section by implementing dynamic version reading from the app bundle. Instead of showing a static "版本1.0.0", the app now displays the actual version and build number in the format "版本vX.X.X(Y)".

  • Replaced hardcoded version string with dynamic bundle reading
  • Added computed property to format version and build information
  • Ensured proper fallback values for missing bundle information

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions
Copy link

Code Coverage Report ❌

Current coverage: 0%

@graycreate graycreate merged commit 0190998 into main Sep 20, 2025
9 checks passed
@graycreate graycreate deleted the bugfix/real-version-display branch September 23, 2025 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants