-
Notifications
You must be signed in to change notification settings - Fork 429
Deprecating sharedProcessPool in SFSDKWebViewStateManager #3933
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- WKProcessPool creation has no effect since iOS 15, so we are no longer using it - sharedProcessPool will be removed in MSDK 14 from SFSDKWebViewStateManager (at which points the deprecation warnings will disappear) - also removed SFSDKWebViewStateManager.h and SFSDKWebViewStateManager.m which are no longer used since the class was converted to swift a while back but were still in the repo
Clang Static Analysis Issues
Generated by 🚫 Danger |
Codecov Report❌ Patch coverage is
❌ Your patch status has failed because the patch coverage (66.66%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## dev #3933 +/- ##
==========================================
- Coverage 63.52% 63.47% -0.05%
==========================================
Files 250 250
Lines 22501 22494 -7
==========================================
- Hits 14293 14278 -15
- Misses 8208 8216 +8
🚀 New features to boost your workflow:
|
| } | ||
|
|
||
|
|
||
| @available(*, deprecated, message: "Deprecated in Salesforce Mobile SDK 13.1 and will be removed in Salesforce Mobile SDK 14.0. WKProcessPool creation has no effect on iOS 15+ and this property will be removed.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should this be 13.2 now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right. I'll send another PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here it is: #3939
sharedProcessPoolwill be removed in MSDK 14 fromSFSDKWebViewStateManager(at which points the deprecation warnings will disappear).WKProcessPoolcreation has no effect since iOS 15, so we are no longer using it.Also removed
SFSDKWebViewStateManager.handSFSDKWebViewStateManager.mwhich should have been removed when the class was converted to Swift a while back.