commit 386a28455aec9abc56348b32e1c71a6a68b76834
parent b2e555284b88ccf3e65dd966d721e8bfe3195f7c
Author: Daniel D’Aquino <daniel@daquino.me>
Date: Fri, 10 Nov 2023 19:52:35 +0000
permissions: Update camera usage string to be more accurate
This changes the text that appears on the camera permission dialog, and makes it clear that camera access is also needed for the scanning of QR codes. This change was made to address a concern raised during the Apple app review process.
Testing
-------
PASS
Device: iPhone 13 Mini (physical)
iOS: 17.1
Damus: This commit
Special remarks: I locally modified the bundle identifier to be able to install a fresh copy of Damus from scratch and go through the prompt, without having to uninstall my own copy of Damus
Steps:
1. Install a fresh copy of Damus
2. On account creation, try to scan an nsec.
3. Make sure the camera usage prompt message is modified to the new string.
Signed-off-by: Daniel D’Aquino <daniel@daquino.me>
Reviewed-by: William Casarin <jb55@jb55.com>
Signed-off-by: William Casarin <jb55@jb55.com>
Diffstat:
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/damus/Info.plist b/damus/Info.plist
@@ -67,7 +67,7 @@
<true/>
</dict>
<key>NSCameraUsageDescription</key>
- <string>Damus needs access to your camera for creating photo posts</string>
+ <string>Damus needs access to your camera in order to upload photos and scan QR codes.</string>
<key>NSAppleMusicUsageDescription</key>
<string>Damus needs access to your media library for playback statuses</string>
<key>NSMicrophoneUsageDescription</key>