Skip to content

fix: add NSBluetoothAlwaysUsageDescription purpose string#518

Merged
piotr-iohk merged 1 commit into
release-2.2.0from
release-2.2.0-fix-purpose-string
Apr 7, 2026
Merged

fix: add NSBluetoothAlwaysUsageDescription purpose string#518
piotr-iohk merged 1 commit into
release-2.2.0from
release-2.2.0-fix-purpose-string

Conversation

@ovitrif
Copy link
Copy Markdown
Collaborator

@ovitrif ovitrif commented Apr 7, 2026

Summary

Stacked on #515

  • Add NSBluetoothAlwaysUsageDescription to Info.plist build settings to resolve App Store Connect warning 90683
  • CoreBluetooth is linked as a transitive dependency of BitkitCore (Rust library) but is not actively used by the app

Test plan

  • Open project in Xcode, verify the key appears in Build Settings > Info.plist Values
  • Build and check the generated Info.plist in build products contains the key
  • Upload new build to App Store Connect and confirm warning 90683 is gone - @piotr-iohk

🤖 Generated with Claude Code

…ng 90683

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@ovitrif ovitrif requested a review from piotr-iohk April 7, 2026 16:31
@ovitrif
Copy link
Copy Markdown
Collaborator Author

ovitrif commented Apr 7, 2026

Self-checks:

  • build in Xcode and run on iPhone

  • Open project in Xcode, verify the key appears in Build Settings > Info.plist Values
Xcode 2026-04-07 005020
  • Build and check the generated Info.plist in build products contains the key
Warp 2026-04-07 005022

@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Apr 7, 2026

Code review

Issue: Missing CHANGELOG.md entry

This is a fix: PR, which per CLAUDE.md requires exactly one changelog entry:

ALWAYS add exactly ONE entry per PR under ## [Unreleased] in CHANGELOG.md for feat: and fix: PRs

The ## [Unreleased] section is currently empty. Please add an entry like:

### Fixed
- Add NSBluetoothAlwaysUsageDescription to resolve App Store Connect warning ITMS-90683 #518

@piotr-iohk
Copy link
Copy Markdown
Collaborator

Upload new build to App Store Connect and confirm warning 90683 is gone - @piotr-iohk

Since we need to update build no to upload, I'll do in base PR.

@piotr-iohk piotr-iohk merged commit c23cf34 into release-2.2.0 Apr 7, 2026
7 of 8 checks passed
@piotr-iohk piotr-iohk deleted the release-2.2.0-fix-purpose-string branch April 7, 2026 16:46
@ovitrif ovitrif added this to the 2.2.0 milestone Apr 7, 2026
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.

2 participants