-
Notifications
You must be signed in to change notification settings - Fork 4
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
Unable to upload to App Store Connect #10
Comments
We have had to revert back to the standard framework package to avoid this bug. This XCFramework is quite literally unusable as-is. Our automated builders are failing to upload to App Store Connect and we cannot rely on developers deleting a duplicate framework within an archive to make the upload succeed. |
I had the same issue and I had to remove the OneSignal Swift package and install it via pod. Make sure to specify the latest version (currently pod 'OneSignal', '3.4.0') because otherwise installs 2.x and has some incompatibilities with the current SDK setup guide. |
We are running into exactly the same issue - we were able to resolve (on CI too) by adding a final build phase script that removes the OneSignal framework from the Plugins folder, as per this technique by Kevin Bui. Update: turned out to not be reliable - also reverting back to OneSignal-iOS-SDK for now. |
Same problem here. Using OneSignal-XCFramework SPM on an M1 Apple Silicon Mac. Xcode 12.5 I just switched over to OneSignal-iOS-SDK for now, still using SPM. Please address this, thank you. |
+1 same issue here. Would like to see this solved. |
Thank you for reporting I am looking into this. It seems like other Binary Swift Packages are having similar issues . I was hopeful that Xcode 12.5 would solve the issue, but it seems it hasn't. To be clear are folks having issues archiving their apps or is there only an issue occurring once the archive has been uploaded to App Store connect? In my testing I am able to successfully archive, but if I have an app extension or app clip that also uses the OneSignal library from the XCFramework Swift Package then validation fails when uploading to app store connect. |
The archive is successful. The error occurs at the end of the upload process and it does not get sent to App Store Connect. |
I am able to reproduce this. I get the same errors as when I include the XCFramework directly and embed it both the App and Extension targets. I am not sure why SwiftPM is trying to embed the framework when it doesn't need to. I will look into ways to modify the Package itself to make it not do that, but otherwise we will need a build script or manual deletion of the framework from the plugin to resolve the issue until Apple addresses it. I apologize for the inconvenience. The manual workaround that works for me is to modify the archive before validating/distributing it.
If you have an app clip or other app extensions that depend on the XCFramework you will need to delete the Frameworks folder from them as well. |
@emawby I've been facing the same issue and ended up here and did the steps you've mentioned. It solved two of the errors, thanks for that but, this still has 2 issues. Could you help me point out what are those? Thanks in advance! |
@imrealashu What version of React-Native-OneSignal are you using? |
+1 facing same issue |
@Semty the second "work around" was actually set by default on a project I am working. I was facing a different problem but triggered by the same root cause (Xcode was crashing at archive export), and it seems like adding the framework into the notification extension target is redundant, at least with Xcode 13.1 as it automatically configures Runpath Search Paths like @emawby described above. |
@emawby's workaround 1 is still useful: with Xcode 13.1 and OneSignalXCFramework 3.9.1 I could archive successfully, but as soon as I started the signing process in an Ad Hoc distribution, Xcode would actually crash! Removing the folder as recommended fixed the crash and therefore the issue. Hopefully, an actual solution that not requires spelunking in the package will be available with a future release. |
Still same issue, I solved the problem with the workaround 1. Thanks @emawby ! .But not sure if it would not affect the behaviour of OneSignal Notifications. I am not able to deliver the notifications to all users (who has accept the push notifications of course). I am using SPM. |
@paskcountry Hello I am sorry to hear about the issue with notification delivery. This issue should not be affecting that so if you are still having issues feel free to open a new issue on the iOS SDK repo. |
Great that is good to know I will remove that line from the documentation |
How to attach keys with SMP? |
Hello, |
I'm still experiencing the same problem ( |
@emawby Please look into this issue! We have all kinds of problems with this with every release we are trying to make!! |
I didn't get this problem the first time I tried in xCode 13.3 - any one else that can verify if this is solved in that version? |
I solved by installing the framework with the CocoaPod (Option 3) instead of using the Swift Package Manager (Option 1). |
This is still an issue regardless of how the extension or app is configured. Using https://github.com/OneSignal/OneSignal-iOS-SDK instead works (Make sure to use at least this version and follow the instructions!) - but this issue needs to be addressed as the documentation for SPM is recommending this version of OneSignal. It appears that the build I linked was actually created to solve this issue, but for some reason, there is no reference to it in this repository or the documentation. Edit: I spoke too soon. It's not fixed. Had to use Carthage to install. |
|
Error with Xcode Version 12.4 (12D4e):
The bundle Payload/Covenant Eyes.app/PlugIns/OneSignal.framework is not contained in a correctly named directory. It should be under "Frameworks".".
Issue is resolved by hand, deleting framework from PlugIns.
https://forums.swift.org/t/swift-package-binary-framework-issue/41922
However, OneSignal is the only troublesome SPM package I've encountered - and we use multiple SPM packages.
Is there anything OneSignal can do to resolve this?
The text was updated successfully, but these errors were encountered: