Skip to content

Fix remote profile update interval unit - #16

Open
H-TTTTT wants to merge 2 commits into
SagerNet:devfrom
H-TTTTT:fix/remote-update-interval-unit
Open

Fix remote profile update interval unit#16
H-TTTTT wants to merge 2 commits into
SagerNet:devfrom
H-TTTTT:fix/remote-update-interval-unit

Conversation

@H-TTTTT

@H-TTTTT H-TTTTT commented Jul 12, 2026

Copy link
Copy Markdown

Summary

  • show a persistent localized min suffix for remote profile update intervals
  • apply the same layout to both new and edit remote profile forms
  • preserve localized suffixes for English, Persian, Russian, Simplified Chinese, and Traditional Chinese

Fixes SagerNet/sing-box#4216

Validation

  • SwiftFormat: all 271 checked files formatted
  • SwiftLint: touched files clean; repository has 18 pre-existing warnings in untouched files
  • localization catalog JSON and translation assertions passed
  • git diff --check

Xcode builds and visual UI tests were not run because the available environment is Linux. The remaining validation gap is layout verification on Apple platforms and compact widths.

@H-TTTTT
H-TTTTT force-pushed the fix/remote-update-interval-unit branch from 6039b6e to 37f21f8 Compare July 28, 2026 02:03
@H-TTTTT

H-TTTTT commented Jul 28, 2026

Copy link
Copy Markdown
Author

Rebased the two feature commits onto the latest dev and removed the unrelated upstream history from this PR. GitHub now reports only the 3 intended files and the branch is cleanly mergeable.

Validation on the updated head (37f21f8):

  • patch content is identical to the previously SwiftFormat/SwiftLint-validated patch (matching stable patch-id)
  • Localizable.xcstrings parses as valid JSON
  • Persian, Russian, Simplified Chinese, and Traditional Chinese min translations verified
  • both profile forms retain the localized, fixed-size min suffix
  • git diff --check passed

Apple-platform builds remain unavailable in this Linux environment.

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.

关于远程配置的自动更新间隔,需要增加时间单位的显示

1 participant