StanfordBDHG / ResearchKit

ResearchKit with Swift Package Manager (SPM), SwiftUI, C++ Interoperability, and visionOS support
https://swiftpackageindex.com/StanfordBDHG/ResearchKit/documentation/researchkitswiftui
Other
7 stars 3 forks source link

Disable SensorKit support by default #18

Closed Supereg closed 8 months ago

Supereg commented 8 months ago

Disable SensorKit support by default

:recycle: Current situation & Problem

As discussed in #17, SensorKit is not available on iPadOS even though it is annotated to be available. This results SensorKit being linked for iOS but the app crashing when run on iPad as the framework cannot be dynamically linked. This issue was induced due to the change in #16 that raised the deployment target to iOS 15. Even conditionally importing SensorKit does not work as it is reported to be available for iPadOS to the compiler.

This PR disables SensorKit integration by default, controlled via the USE_SENSOR_KIT macro. While the ORKSensorPermissionType is still there it shows up as unavailable for the platform in ResearchKit UI.

:gear: Release Notes

:books: Documentation

--

:white_check_mark: Testing

Verified locally.

Code of Conduct & Contributing Guidelines

By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines: