-
Notifications
You must be signed in to change notification settings - Fork 12
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
Release: 1.8.0 #84
Release: 1.8.0 #84
Changes from 11 commits
2efb7ab
c657f2f
674c68c
d38b847
6f07ffe
05f1890
90a50b3
0d1362c
e8dbd05
6b1fe47
410b541
87097a7
9393fd3
17f9fc9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -19,7 +19,7 @@ executors: | |
ios-executor: | ||
working_directory: ~/ios | ||
macos: | ||
xcode: 12.0 | ||
xcode: 12.5.1 | ||
|
||
jobs: | ||
test-ios: | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -7,8 +7,8 @@ GEM | |
artifactory (3.0.15) | ||
atomos (0.1.3) | ||
aws-eventstream (1.1.1) | ||
aws-partitions (1.474.0) | ||
aws-sdk-core (3.116.0) | ||
Comment on lines
-10
to
-11
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. updated fastlane |
||
aws-partitions (1.478.0) | ||
aws-sdk-core (3.117.0) | ||
aws-eventstream (~> 1, >= 1.0.2) | ||
aws-partitions (~> 1, >= 1.239.0) | ||
aws-sigv4 (~> 1.1) | ||
|
@@ -20,7 +20,7 @@ GEM | |
aws-sdk-core (~> 3, >= 3.112.0) | ||
aws-sdk-kms (~> 1) | ||
aws-sigv4 (~> 1.1) | ||
aws-sigv4 (1.2.3) | ||
aws-sigv4 (1.2.4) | ||
aws-eventstream (~> 1, >= 1.0.2) | ||
babosa (1.0.4) | ||
claide (1.0.3) | ||
|
@@ -36,7 +36,7 @@ GEM | |
dotenv (2.7.6) | ||
emoji_regex (3.2.2) | ||
excon (0.84.0) | ||
faraday (1.5.0) | ||
faraday (1.5.1) | ||
faraday-em_http (~> 1.0) | ||
faraday-em_synchrony (~> 1.0) | ||
faraday-excon (~> 1.1) | ||
|
@@ -54,7 +54,7 @@ GEM | |
faraday-excon (1.1.0) | ||
faraday-httpclient (1.0.1) | ||
faraday-net_http (1.0.1) | ||
faraday-net_http_persistent (1.1.0) | ||
faraday-net_http_persistent (1.2.0) | ||
faraday-patron (1.0.0) | ||
faraday_middleware (1.0.0) | ||
faraday (~> 1.0) | ||
|
@@ -122,7 +122,7 @@ GEM | |
google-cloud-env (1.5.0) | ||
faraday (>= 0.17.3, < 2.0) | ||
google-cloud-errors (1.1.0) | ||
google-cloud-storage (1.34.0) | ||
google-cloud-storage (1.34.1) | ||
addressable (~> 2.5) | ||
digest-crc (~> 0.4) | ||
google-apis-iamcredentials_v1 (~> 0.1) | ||
|
@@ -154,7 +154,7 @@ GEM | |
os (1.1.1) | ||
plist (3.6.0) | ||
public_suffix (4.0.6) | ||
rake (13.0.4) | ||
rake (13.0.6) | ||
representable (3.1.1) | ||
declarative (< 0.1.0) | ||
trailblazer-option (>= 0.1.1, < 0.2.0) | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -20,6 +20,8 @@ import com.revenuecat.purchases.identifyWith | |
import com.revenuecat.purchases.purchasePackageWith | ||
import com.revenuecat.purchases.purchaseProductWith | ||
import com.revenuecat.purchases.resetWith | ||
import com.revenuecat.purchases.logInWith | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. from this point on, the changes had been there merged before, but were removed in #75, because identity v3 wasn't ready to go in the native SDKs |
||
import com.revenuecat.purchases.logOutWith | ||
import com.revenuecat.purchases.restorePurchasesWith | ||
import com.revenuecat.purchases.common.PlatformInfo | ||
import com.revenuecat.purchases.interfaces.Callback | ||
|
@@ -182,6 +184,27 @@ fun restoreTransactions( | |
} | ||
} | ||
|
||
fun logIn( | ||
appUserID: String, | ||
onResult: OnResult | ||
) { | ||
Purchases.sharedInstance.logInWith(appUserID, | ||
onError = { onResult.onError(it.map()) }, | ||
onSuccess = { purchaserInfo, created -> | ||
val resultMap: Map<String, Any?> = mapOf( | ||
"purchaserInfo" to purchaserInfo.map(), | ||
"created" to created | ||
) | ||
onResult.onReceived(resultMap) | ||
}) | ||
} | ||
|
||
fun logOut(onResult: OnResult) { | ||
Purchases.sharedInstance.logOutWith(onError = { onResult.onError(it.map()) }) { | ||
onResult.onReceived(it.map()) | ||
} | ||
} | ||
|
||
fun reset( | ||
onResult: OnResult | ||
) { | ||
|
@@ -305,7 +328,7 @@ private fun getPurchaseCompletedFunction(onResult: OnResult): (Purchase?, Purcha | |
return { purchase, purchaserInfo -> | ||
onResult.onReceived( | ||
mapOf( | ||
"productIdentifier" to purchase?.sku, | ||
"productIdentifier" to purchase?.skus, | ||
Comment on lines
-308
to
+347
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. related to changes in RevenueCat/purchases-android#335 |
||
"purchaserInfo" to purchaserInfo.map() | ||
) | ||
) | ||
|
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.
we were getting
definition conflicts with previous value
errors pointing towards the eTags stuff inpurchases-ios
for some reason.it looks like it was either a bug in Xcode 12.0, or in one of the other dependencies associated with the CircleCI instance for
xcode 12.0
. In any case, it gets fixed by bumping the version number here, and it works correctly locally.