-
Notifications
You must be signed in to change notification settings - Fork 315
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
PurchaseTester
: fixed watchOS
build and ASC deployment
#2701
Conversation
PurchaseTester
: fixed watchOS
buildPurchaseTester
: fixed watchOS
build and ASC deployment
Fixes https://app.circleci.com/pipelines/github/RevenueCat/purchases-ios/12159/workflows/89db4b5d-bafc-4b99-ac86-4f205e49f1bf/jobs/81368 We were not correctly embedding (and therefore building) the `watchOS` extension until #2685. That exposed a bunch of other issues which this fixes.
cdc3c8a
to
d1949e0
Compare
Codecov Report
@@ Coverage Diff @@
## main #2701 +/- ##
==========================================
- Coverage 86.58% 86.57% -0.01%
==========================================
Files 211 211
Lines 14987 14987
==========================================
- Hits 12976 12975 -1
- Misses 2011 2012 +1 |
@@ -790,13 +792,13 @@ | |||
"PROVISIONING_PROFILE_SPECIFIER[sdk=iphoneos*]" = "match AppStore com.revenuecat.sampleapp"; | |||
"PROVISIONING_PROFILE_SPECIFIER[sdk=macosx*]" = "match AppStore com.revenuecat.sampleapp macos"; | |||
"PROVISIONING_PROFILE_SPECIFIER[sdk=watchos*]" = "match AppStore com.revenuecat.sampleapp"; | |||
SUPPORTED_PLATFORMS = "appletvos appletvsimulator iphoneos iphonesimulator macosx watchos watchsimulator xros xrsimulator"; |
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.
The main target was configured as compatible with watchOS
, which didn't make sense.
<TestableReference | ||
skipped = "NO"> | ||
<BuildableReference | ||
BuildableIdentifier = "primary" | ||
BlueprintIdentifier = "2CD2C501278C9B02005D1CC2" | ||
BuildableName = "Tests iOS.xctest" | ||
BlueprintName = "Tests iOS" | ||
ReferencedContainer = "container:PurchaseTester.xcodeproj"> | ||
</BuildableReference> | ||
</TestableReference> |
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.
There is no test target, so this was a dead link.
**This is an automatic release.** ### Bugfixes * Fix google play purchases missing purchase date (#2703) via Toni Rico (@tonidero) ### Other Changes * `PurchaseTester`: fixed `watchOS` build and ASC deployment (#2701) via NachoSoto (@NachoSoto) * Add `Data.sha1` (#2696) via NachoSoto (@NachoSoto) * Refactor: extract `ErrorResponse` into its own file (#2697) via NachoSoto (@NachoSoto) * Add `Sequence<AdditiveArithmetic>.sum()` (#2694) via NachoSoto (@NachoSoto) * Refactored `Data.asString` implementation (#2695) via NachoSoto (@NachoSoto) * `Diagnostics`: new `FileHandler` for abstracting file operations (#2673) via NachoSoto (@NachoSoto)
### Description While uploading purchase tester to testflight, we were getting these errors: ``` [Application Loader Error Output]: ERROR: [ContentDelivery.Uploader] Asset validation failed (90396) Invalid Icon. The watch application 'PurchaseTester.app/Watch/PurchaseTesterWatchOS.app' contains an icon file 'Icon [email protected]' with an alpha channel. Icons should not have an alpha channel. [Application Loader Error Output]: ERROR: [ContentDelivery.Uploader] Asset validation failed (90717) Invalid App Store Icon. The App Store Icon in the asset catalog in 'PurchaseTester.app/Watch/PurchaseTesterWatchOS.app' can't be transparent nor contain an alpha channel. ``` This icon was added in #2701. After trying it manually, removing the alpha channel seems to have worked in fixing the issue while uploading. This PR just changes that icon with a version without an alpha channel.
Fixes https://app.circleci.com/pipelines/github/RevenueCat/purchases-ios/12159/workflows/89db4b5d-bafc-4b99-ac86-4f205e49f1bf/jobs/81368
We were not correctly embedding (and therefore building) the
watchOS
extension until #2685. That exposed a bunch of other issues which this fixes.