Skip to content

fix: require iOS 12 for OneSignal iOS 5.6.0 - #896

Merged
fadi-george merged 2 commits into
mainfrom
fix/ios-5.6-release-compatibility
Sep 9, 2026
Merged

fix: require iOS 12 for OneSignal iOS 5.6.0#896
fadi-george merged 2 commits into
mainfrom
fix/ios-5.6-release-compatibility

Conversation

@fadi-george

Copy link
Copy Markdown
Collaborator

Description

One Line Summary

Align Unity's iOS minimum deployment target with OneSignal iOS 5.6.0.

Details

Motivation

OneSignalXCFramework 5.6.0 requires iOS 12, while the generated Notification Service Extension still targeted iOS 11.

Scope

Raises the generated extension target to iOS 12 and updates the matching requirements documentation.

Testing

No tests were added because this changes generated project configuration and documentation.

Validated with:

  • csharpier check .
  • python3 .github/scripts/validate-unity-distribution.py

Affected code checklist

  • Notifications
    • Push Processing
  • Public API changes

Checklist

Overview

  • I have filled out all REQUIRED sections above
  • PR does one thing
  • No public API changes

Testing

  • Test coverage is added or explained
  • All applicable automated checks pass
  • Device testing is not applicable to deployment metadata

Final pass

  • Code is as readable as possible
  • I reviewed this PR

@fadi-george
fadi-george requested a review from a team as a code owner September 8, 2026 21:53
Base automatically changed from rel/5.3.4 to main September 9, 2026 16:17
Co-authored-by: Cursor <cursoragent@cursor.com>
@fadi-george
fadi-george force-pushed the fix/ios-5.6-release-compatibility branch from 745eec7 to 1c096a0 Compare September 9, 2026 22:52
@fadi-george
fadi-george merged commit e37e2e9 into main Sep 9, 2026
1 check passed
@fadi-george
fadi-george deleted the fix/ios-5.6-release-compatibility branch September 9, 2026 22:54
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.

3 participants