-
Notifications
You must be signed in to change notification settings - Fork 50
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
[WIP] PoC: Emergetools snapshot testing #1359
Closed
Closed
Changes from all commits
Commits
Show all changes
168 commits
Select commit
Hold shift + click to select a range
e07f94b
Create paywall tester app (#1218)
tonidero c170c6d
Add PaywallData and its classes (#1219)
vegaro f617e36
Add RevenueCatUI module and initial API (#1213)
tonidero a1987db
Adds kotlinx serialization (#1221)
vegaro 7fd52a8
Serializes PaywallData (#1222)
vegaro 208385a
PaywallTester: Visualize offerings in offerings tab and navigate to n…
tonidero 0377b53
Paywalls: Add simple paywall and use in tester app (#1223)
tonidero f177873
Paywalls: Use coroutines to interact with SDK (#1224)
tonidero 051b591
Add revision and remove default_locale (#1226)
vegaro 408b404
Add configForLocale to PaywallData (#1227)
vegaro 2d89728
Fix tests that broke when adding PaywallData (#1229)
vegaro b98cdf6
Fix OfferingsParser exceptions being swallowed (#1228)
vegaro 666df9e
Paywalls: Use paywall data in paywall (#1230)
tonidero 19a2b93
Nightly deploy of paywall tester (#1231)
vegaro ab9ce1f
offerdetails is optional
vegaro c611ef6
Add callToActionSecondaryBackground (#1236)
vegaro 1a183c4
Paywalls: Add initial Template2 UI and using colors and texts in payw…
tonidero c02c2d2
Paywalls: Add logger class to unify logging behavior (#1233)
tonidero 0453d1a
Paywalls: Use coil to load images (#1235)
tonidero baca8d4
Add PaywallDataTest (#1237)
vegaro f5eeace
Paywalls: Organize files into packages (#1241)
tonidero 0185c77
`Paywalls`: created `PaywallBackground` (#1240)
NachoSoto e1c78a0
Hook up purchasing listeners (#1243)
vegaro c313806
Show dialog after completed or error purchasing (#1246)
vegaro 6c3867c
`Paywalls`: new `TestData` to store fake paywalls (#1239)
NachoSoto a463ccd
Paywalls: Map data inputs into consumable TemplateConfiguration class…
tonidero 21159a8
PaywallTester: Do not show error dialog if cancellation (#1252)
tonidero c18360d
PaywallTester: Improve AppInfo screen to allow LogIn/LogOut and displ…
tonidero ed76366
Add `valueInMonths` helper to `Period` class (#1250)
tonidero 08106d2
Add `formattedPricePerMonth` helper to `StoreProduct` and `PricingPha…
tonidero d71869b
Paywalls: Add variable processing to paywall strings (#1251)
tonidero c83108a
`Paywalls`: extracted `ApplicationContext` interface and `MockViewMod…
NachoSoto f1ce20c
Paywalls: Add template configuration factory tests (#1249)
tonidero bd25bf4
Adds localizedConfiguration (#1238)
vegaro b1c2716
Paywalls: Add more tests in `VariableProcessorTest` (#1262)
tonidero 3d7deb1
Paywalls: Add `product_name` variable support (#1263)
tonidero cefc0cc
Paywalls: Add support for `sub_period` variable (#1264)
tonidero 8cf2d19
Created `Footer` (#1245)
NachoSoto 61c96ea
`Paywalls`: initial template 1 implementation (#1259)
NachoSoto a794347
`Paywalls`: added transition when loading images (#1272)
NachoSoto 6b68606
Paywalls: Adds some initial spanish strings and some initial preview …
tonidero 32ba3fd
Paywalls: Fix locale selection logic for previews (#1267)
tonidero 5ebbb04
Paywalls: Implement `price_per_period` variable support (#1269)
tonidero 57b155f
Paywalls: Add support for SUB_DURATION variable in paywalls (#1276)
tonidero 1da7177
Created enum with variable names (#1270)
vegaro a176850
Paywalls: Add tests for additional periods in variable processor (#1278)
tonidero 00f8ca1
`Paywalls`: new `PaywallIcon` (#1274)
NachoSoto 19faed4
Paywalls: Add support for `sub_offer_price` variable processing in pa…
tonidero 2987fce
Paywalls: Add support for `sub_offer_duration` variable in paywalls (…
tonidero a4ecd77
Paywalls: Add support for `total_price_and_per_month` variable in pay…
tonidero dda3534
`Paywalls`: `InternalPaywallView` now takes a `PaywallViewMode` param…
NachoSoto af717d8
Creates default PaywallData (#1261)
vegaro 2ea162f
Paywalls: Fix state update upon locale changes (#1287)
tonidero dc1d023
PaywallData validation (#1273)
vegaro efc3c51
Add preview for default data (#1292)
vegaro c119973
Template 3 (#1294)
vegaro 4885224
Better handling of packages not found for id error (#1295)
vegaro 1364f32
Merge branch 'main' of github.com:RevenueCat/purchases-android into p…
tonidero 42cad0b
`Paywalls`: Intro Eligibility dependent composable (#1286)
NachoSoto bc072d1
Paywalls: Add multiple offers fields to the paywall response and proc…
tonidero 13af808
Paywalls: Add API to display paywall as a composable dialog (#1297)
tonidero 37e331f
Paywalls: Template2 improve select button (#1301)
tonidero dab8cbe
Paywalls: Add some tests to intro eligibility calculation (#1303)
tonidero cc1e396
Extract templates from TestData (#1299)
vegaro 22405a5
Refactor checking for available packages when creating package config…
vegaro 6063a51
`Paywalls`: changed `PurchaseButton` to use `IntroEligibilityStateVie…
NachoSoto d6b1c7c
Fix template test data (#1308)
vegaro 220631f
`Paywalls`: created `LoadingPaywallView` (#1282)
NachoSoto 26b2e0c
`Paywalls`: implemented `LoadingPaywallView` with `placeholder` (#1284)
NachoSoto 946bb9e
PaywallData validation tests (#1310)
vegaro c74f386
`Paywalls`: use `IntroEligibilityStateView` (#1311)
NachoSoto 4b105c8
Paywalls: Add PaywallFooter composable to present a minified paywall …
tonidero 58c3502
Paywalls: Update paywall tester to be able to display paywall footer …
tonidero 02b1eed
`PaywallsTester`: added template name to offerings list (#1316)
NachoSoto b2ffbf3
`Paywalls`: Markdown support (#1312)
NachoSoto 8cfb938
`Paywalls`: fixed `PaywallDialog.setRequiredEntitlementIdentifier` (#…
NachoSoto 0cb8c8e
`Paywalls`: changed PaywallsTester app icon (#1323)
NachoSoto 6912c9e
Parse `{{ sub_offer_price_2 }}` and `{{ sub_offer_duration_2 }}` vari…
vegaro e44d4ce
Paywalls: Add support to launch paywall as activity (#1317)
tonidero 1a5cf15
Fix material theme references to use Material3 versions (#1326)
tonidero 5aa3d80
Add support for multiple intro offers in IntroEligibilityStateView (#…
vegaro 1d96bbc
Rename packages to packageIds in PaywallData (#1309)
vegaro ec89c39
Log errors when displaying default paywall (#1318)
vegaro b76da03
`Paywalls`: log error when failing to load images (#1321)
NachoSoto 61784d1
Add revenuecatui gradle.properties to specify name of dependency (#1324)
tonidero 81f8c1d
Paywalls: Make DialogScaffold private (#1329)
tonidero 5db0e8a
`Paywalls`: `PaywallColor` supports RGBA (#1332)
NachoSoto fa43b05
Expose new `TestStoreProduct` (#1333)
NachoSoto b2f0571
`Paywalls`: polished `Template2`, `PurchaseButton`, and `Footer` (#1336)
NachoSoto 98fed28
Add sample templates to paywalls tester (#1330)
vegaro a3baf26
`Paywalls`: animate package selection transitions (#1337)
NachoSoto 84b0a47
`Paywalls`: display default template in paywalls tester (#1341)
NachoSoto 0651eb6
`Paywalls`: improved default paywall (#1342)
NachoSoto f1e311a
Fix loading another template in Paywalls screen (#1345)
vegaro 39db390
`Paywalls`: polish template 3 (#1344)
NachoSoto a85a918
`Paywalls`: polish template 1 (#1343)
NachoSoto 1a9688a
Update to use name instead of id when creating sample offering (#1347)
vegaro 9721a5f
`Paywalls`: `ViewThatFits` equivalent to improve `Footer` (#1258)
NachoSoto 54b36b5
Paywalls: Add restore paywall callbacks (#1350)
tonidero e03a270
`Paywalls`: new `PaywallActivityLauncher.launchIfNeeded` methods (#1335)
NachoSoto 41df0a7
`Paywalls`: disallow purchasing currently subscribed products (#1334)
NachoSoto dcb157e
`Paywalls`: support `{{ sub_duration_in_months }}` (#1348)
NachoSoto 5a279c6
`Paywalls`: handle "action in progress" state (#1346)
NachoSoto 865a6c2
`Paywalls`: enable library publishing (#1353)
NachoSoto f395532
`Paywalls`: disable `PurchaseButton` during purchases (#1352)
NachoSoto d87db9b
Paywalls: Rename `PaywallView` to `Paywall` (#1351)
tonidero 8f9620f
`Paywalls`: fixed `Footer` padding (#1354)
NachoSoto 1ca3aff
Merge branch 'main' into paywalls
NachoSoto 39ab56c
Paywalls: Support custom fonts through FontProvider (#1328)
tonidero b521669
Paywalls: Support Google fonts and font families with multiple fonts …
tonidero 6801f5c
Paywalls: Add custom font example to paywalls screen (#1358)
tonidero 402798d
Paywalls: Fix proguard rules kotlinx serialization (#1356)
tonidero 19ab46b
Paywalls: Fix paywall compose previews (#1360)
tonidero a16a6d8
`Paywalls`: improve `LoadingPaywall` (#1364)
NachoSoto c2f6627
`Paywalls`: `PaywallViewModel` tests (#1357)
NachoSoto 1d79340
Paywalls: backwards compatible blurring (#1327)
aboedo 5c7b87d
`Paywalls`: finished localization support (#1362)
NachoSoto f8acc77
Paywalls: Support condensed footer presentation in template 2 (#1365)
tonidero 5466268
Add `StoreProduct.pricePerMonth` (#1369)
NachoSoto 355f6d4
Paywalls: Support footer in template 1 (#1366)
tonidero 6ec7973
`Paywalls`: improve handling of lifetime/custom packages (#1363)
NachoSoto d7d9b57
Paywalls: Add offer details in template 2 when in condensed form (#1371)
tonidero 7a15c1e
`Paywalls`: calculate `discountRelativeToMostExpensivePerMonth` (#1370)
NachoSoto b3b41ae
Paywalls: Support footer in template 3 (#1367)
tonidero 088bbe0
Paywalls: Enable footer modes in paywall tester paywalls tab (#1368)
tonidero 848b5ab
`Paywalls`: optimize `PurchaseButton` (#1376)
NachoSoto 757cf46
`Paywalls`: improve `FooterDialog` corner radius (#1374)
NachoSoto e0f78d4
`Paywalls`: optimize `AdaptiveComposable` (#1377)
NachoSoto 52fd5da
`Paywalls`: `PurchaseButton` supports gradients (#1378)
NachoSoto 6a641d7
Paywalls: Predownload offering images if paywalls sdk exists (#1372)
tonidero 064f4bc
`Paywalls`: optimize backwards compatible blurring (#1379)
NachoSoto 175f1a7
`Paywalls`: polish template 2 spacing (#1381)
NachoSoto 4f45a9d
`Paywalls`: fix backwards-compatible blurring of default paywall back…
NachoSoto 2228907
`Paywalls`: fixed incorrect background on footer (#1382)
NachoSoto 58ceeac
`Paywalls`: improve error handling (#1383)
NachoSoto 50244e5
`Paywalls`: display purchase/restore errors (#1384)
NachoSoto 41d88ef
`Paywalls`: support for Markdown links (#1387)
NachoSoto 035baff
`Paywalls`: use `LocalUriHandler` for opening links (#1388)
NachoSoto 0273661
`Paywalls`: increase `PaywallBackground` blur radius to match iOS
NachoSoto c59c681
`Paywalls`: fixed accessibility across templates (#1392)
NachoSoto a922112
`Paywalls`: `PaywallDialogOptions` no longer requires `dismissRequest…
NachoSoto bdf8c60
`Paywalls`: fix template 3 offer details color (#1394)
NachoSoto 26d5832
`Paywalls`: template 4 (#1349)
NachoSoto 2436263
`Paywalls`: improve template 2 layout (#1396)
NachoSoto 65ada91
Paywalls: Add experimental annotation to all public APIs in RevenueCa…
tonidero 5ab6631
Fix navigation after closing paywall in Paywalls screen (#1403)
vegaro ee14d63
Paywalls: Allow trigger manual paywall tester release (#1406)
tonidero b1d109f
`Paywalls`: decode empty images as `null`
NachoSoto 4984a75
`Paywalls`: `RemoteImage` always renders a placeholder on previews
NachoSoto 236f8cb
Add test for packageInfo.localizedDiscount (#1407)
vegaro d90987d
Fix button padding in Loading paywall (#1405)
vegaro c63197e
Animate transition between loading and loaded paywall (#1404)
vegaro 2b494b7
`Paywalls`: ignore URL deserialization errors (#1399)
NachoSoto 2078bbf
Fix onDismiss getting called twice in PaywallDialogs (#1408)
vegaro e84b89d
`Paywalls`: fix `MarkdownText` stealing touches from package buttons …
NachoSoto c240b33
Paywalls: Add close button option (#1390)
tonidero 2767fb9
`Paywalls`: improve template 4 A11y support
NachoSoto 0010b51
`Paywalls`: footer view should not render background image
NachoSoto 30064ba
`Paywalls`: convert empty strings to `null`
NachoSoto 318f5ca
`Paywalls`: `PaywallData` errors shouldn't make `Offering`s fail to d…
NachoSoto 3778333
`Paywalls`: support `text3` and `accent3` colors (#1409)
NachoSoto 2ee70be
Paywalls: Fix purchase button text flashing when changing selected pa…
tonidero d1329ed
Paywalls: Add bundled font to paywall tester (#1415)
tonidero f64e7fa
`Paywalls`: update template 4 colors (#1410)
NachoSoto 12d4446
Paywalls: Fix wrong view model data after configuration changes (#1417)
tonidero 8997509
`Paywalls`: implemented template 5 (#1412)
NachoSoto c21a8df
PoC for Emergetools snapshot testing
tonidero cc0b9e8
Go back to java 1.8 for now
tonidero File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
17 changes: 17 additions & 0 deletions
17
api-tester/src/main/java/com/revenuecat/apitester/java/TestStoreProductAPI.java
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
package com.revenuecat.apitester.java; | ||
|
||
import com.revenuecat.purchases.models.Period; | ||
import com.revenuecat.purchases.models.Price; | ||
import com.revenuecat.purchases.models.TestStoreProduct; | ||
|
||
@SuppressWarnings({"unused"}) | ||
final class TestStoreProductAPI { | ||
static void check(final Price price, final Period period) { | ||
new TestStoreProduct( | ||
"id", "title", "description", price, period, null, null | ||
); | ||
new TestStoreProduct( | ||
"id", "title", "description", price, period, period, price | ||
); | ||
} | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
42 changes: 42 additions & 0 deletions
42
api-tester/src/main/java/com/revenuecat/apitester/kotlin/TestStoreProductAPI.kt
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,42 @@ | ||
package com.revenuecat.apitester.kotlin | ||
|
||
import com.revenuecat.purchases.models.Period | ||
import com.revenuecat.purchases.models.Price | ||
import com.revenuecat.purchases.models.TestStoreProduct | ||
|
||
@Suppress("unused") | ||
private class TestStoreProductAPI { | ||
fun checkConstructor(price: Price, period: Period?) { | ||
TestStoreProduct( | ||
id = "ID", | ||
title = "title", | ||
price = price, | ||
description = "description", | ||
period = period, | ||
) | ||
TestStoreProduct( | ||
id = "ID", | ||
title = "title", | ||
price = price, | ||
description = "description", | ||
period = period, | ||
freeTrialPeriod = period, | ||
) | ||
TestStoreProduct( | ||
id = "ID", | ||
title = "title", | ||
price = price, | ||
description = "description", | ||
period = period, | ||
introPrice = null, | ||
) | ||
TestStoreProduct( | ||
id = "ID", | ||
title = "title", | ||
price = price, | ||
description = "description", | ||
period = period, | ||
introPrice = price, | ||
) | ||
} | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
/build |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,96 @@ | ||
plugins { | ||
alias(libs.plugins.android.application) | ||
alias(libs.plugins.kotlin.android) | ||
alias(libs.plugins.ksp) | ||
alias(libs.plugins.emergetools) | ||
} | ||
|
||
emerge { | ||
// Emerge uses the EMERGE_API_TOKEN env variable by default, so no need to set this explicitly, this is just for example purposes | ||
apiToken.set(System.getenv("EMERGE_API_TOKEN")) | ||
} | ||
|
||
apply from: "$rootProject.projectDir/base-application.gradle" | ||
|
||
android { | ||
namespace 'com.revenuecat.paywallstester' | ||
|
||
defaultConfig { | ||
applicationId "com.revenuecat.paywall_tester" | ||
minSdk 24 | ||
versionCode paywallTesterVersionCode as Integer | ||
versionName paywallTesterVersionName | ||
|
||
// Library modules have a dimension used to separate different apis. | ||
// Our applications however don't need the extra flavor. This makes sure that we use the | ||
// default flavor. | ||
missingDimensionStrategy 'apis', 'defaults' | ||
|
||
vectorDrawables { | ||
useSupportLibrary true | ||
} | ||
} | ||
|
||
signingConfigs { | ||
release { | ||
storeFile file('keystore') | ||
storePassword project.properties['releaseKeystorePassword'] | ||
keyAlias project.properties['releaseKeyAlias'] | ||
keyPassword project.properties['releaseKeyPassword'] | ||
} | ||
} | ||
|
||
buildTypes { | ||
release { | ||
minifyEnabled false | ||
proguardFiles getDefaultProguardFile('proguard-android-optimize.txt'), 'proguard-rules.pro' | ||
signingConfig signingConfigs.release | ||
} | ||
} | ||
compileOptions { | ||
sourceCompatibility JavaVersion.VERSION_1_8 | ||
targetCompatibility JavaVersion.VERSION_1_8 | ||
} | ||
kotlinOptions { | ||
jvmTarget = JavaVersion.VERSION_1_8.toString() | ||
} | ||
buildFeatures { | ||
compose true | ||
} | ||
composeOptions { | ||
kotlinCompilerExtensionVersion '1.4.0' | ||
} | ||
packaging { | ||
resources { | ||
excludes += '/META-INF/{AL2.0,LGPL2.1}' | ||
} | ||
} | ||
} | ||
|
||
dependencies { | ||
implementation project(path: ':purchases') | ||
implementation project(path: ':feature:amazon') | ||
implementation project(path: ':ui:debugview') | ||
implementation project(path: ':ui:revenuecatui') | ||
|
||
implementation libs.androidx.core | ||
implementation platform(libs.kotlin.bom) | ||
implementation libs.androidx.lifecycle.runtime.ktx | ||
implementation platform(libs.compose.bom) | ||
implementation libs.material | ||
implementation libs.compose.ui | ||
implementation libs.compose.ui.graphics | ||
implementation libs.compose.ui.tooling.preview | ||
implementation libs.compose.material3 | ||
implementation libs.compose.material | ||
implementation libs.compose.window.size | ||
implementation libs.navigation.compose | ||
implementation libs.compose.ui.google.fonts | ||
debugImplementation libs.compose.ui.tooling | ||
debugImplementation libs.androidx.test.compose.manifest | ||
|
||
implementation "com.emergetools.snapshots:snapshots-annotations:0.8.3" | ||
|
||
androidTestImplementation "com.emergetools.snapshots:snapshots:0.8.3" | ||
ksp "com.emergetools.snapshots:snapshots-processor:0.8.3" | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,21 @@ | ||
# Add project specific ProGuard rules here. | ||
# You can control the set of applied configuration files using the | ||
# proguardFiles setting in build.gradle. | ||
# | ||
# For more details, see | ||
# http://developer.android.com/guide/developing/tools/proguard.html | ||
|
||
# If your project uses WebView with JS, uncomment the following | ||
# and specify the fully qualified class name to the JavaScript interface | ||
# class: | ||
#-keepclassmembers class fqcn.of.javascript.interface.for.webview { | ||
# public *; | ||
#} | ||
|
||
# Uncomment this to preserve the line number information for | ||
# debugging stack traces. | ||
#-keepattributes SourceFile,LineNumberTable | ||
|
||
# If you keep the line number information, uncomment this to | ||
# hide the original source file name. | ||
#-renamesourcefileattribute SourceFile |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Had to update to the stable version since it's using kotlin 1.8.0 now instead of 1.7.21 as we were before for some reason. Also need to investigate this.