Skip to content

Mirror HealthKit update usage description in InfoPlist.strings - #5269

Merged
bgoncal merged 1 commit into
mainfrom
fix/healthkit-usage-string-localization
Jul 28, 2026
Merged

Mirror HealthKit update usage description in InfoPlist.strings#5269
bgoncal merged 1 commit into
mainfrom
fix/healthkit-usage-string-localization

Conversation

@bgoncal

@bgoncal bgoncal commented Jul 28, 2026

Copy link
Copy Markdown
Member

AI Policy

Select exactly one option that describes AI usage in this contribution:

  • I have not used AI for this contribution.
  • AI assistance was used for this contribution.
  • AI fully generated the code for this contribution, but I've reviewed and understood it before submitting and will respond without AI during review.

Summary

Follow-up to #5267, which added NSHealthUpdateUsageDescription to Info.plist but not to en.lproj/InfoPlist.strings. This adds the matching entry so the HealthKit purpose string stays consistent with Info.plist and remains translatable, alongside the other sensor purpose strings (Health, Motion, Focus) already mirrored there.

Screenshots

No visual change.

Link to pull request in Documentation repository

Documentation: home-assistant/companion.home-assistant#

Any other notes

Follow-up to #4923 and #5267. Scoped to the HealthKit key only; a few other purpose strings (Bluetooth, Face ID, Reminders) are also unmirrored but are pre-existing and left for a separate cleanup.

Copilot AI review requested due to automatic review settings July 28, 2026 22:57

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 updates the app’s localized InfoPlist.strings to include the missing HealthKit update purpose string so the NSHealthUpdateUsageDescription text remains translatable and consistent with Info.plist.

Changes:

  • Add NSHealthUpdateUsageDescription to Sources/App/Resources/en.lproj/InfoPlist.strings.

Comment thread Sources/App/Resources/en.lproj/InfoPlist.strings
@bgoncal
bgoncal enabled auto-merge (squash) July 28, 2026 23:12
@bgoncal
bgoncal merged commit 6ff85c5 into main Jul 28, 2026
12 checks passed
@bgoncal
bgoncal deleted the fix/healthkit-usage-string-localization branch July 28, 2026 23:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants