Eng: Kmp provided help center feature. Experimental

.github/workflows/umbrella.yml

@@ -4,11 +4,7 @@ on:
   push:
     branches:
       - develop
-      - makerdays/chat-claim
-    paths:
-      - 'app/authlib/**'
-      - 'app/feature/feature-claim-chat/**'
-      - 'app/shareddi/**'
+      - feat/help-center-kmp
   workflow_dispatch:
 
 concurrency:

app/apollo/apollo-octopus-public/src/commonMain/graphql/com/hedvig/android/apollo/octopus/schema.graphqls

@@ -1728,6 +1728,11 @@ type ExternalInsurer {
   displayName: String!
   insurelyId: String
 }
+type FetchedExternalInsurance {
+  displayName: String!
+  subtitle: String
+  insurer: ExternalInsurer!
+}
 type FirstVetAction {
   sections: [FirstVetSection!]!
 }
@@ -2729,6 +2734,22 @@ type MemberPaymentAvailablePaymentMethod {
    True if the member can set up this payment method for payout. 
   """
   supportsPayout: Boolean!
+  """
+  True if this method is already ACTIVE for member and can be chosen as default directly without setup, false if
+  this is a new payment method that the member has not yet set up.
+  If this is true, then the `details` field will be populated with the payment method details. If this is false, then
+  the `details` field will be null since the member has not yet set up this payment method.
+  If true then this method can be set up as default directly by calling `paymentMethodSetDefaultPayin` or
+  `paymentMethodSetDefaultPayout` mutation depending on if it's a payin or payout method. If false, then the
+  corresponding mutation for setting up this payment method should be called, eg. `paymentMethodSetupTrustly`,
+  `paymentMethodSetupSwishPayin` etc.
+  """
+  isActive: Boolean!
+  """
+  For already connected and ACTIVE methods, ie isActive=true, specific details of the actual connection - e.g. a bank
+  account reference, phone number for swish, or email/kivra for invoice.
+  """
+  details: PaymentMethodDetails
 }
 type MemberPaymentChargeMethodInfo {
   """
@@ -2801,48 +2822,75 @@ type MemberPaymentInformation {
 }
 type MemberPaymentMethod {
   """
-   The unique id of the payment method. This id is used for switching default and revoking payment methods. 
-  """
-  id: ID!
-  """
-   Payment provider, eg Trustly, Swish, Nordea, Kivra etc. 
+  Payment provider, eg Trustly, Swish, Nordea, Kivra etc.
+  This is used as the "identifier" of the payment method since there can only be one ACTIVE or PENDING payment method
+  per provider.
   """
   provider: MemberPaymentProvider!
   """
-  The payment method status - ACTIVE, PENDING, or PENDING_DEFAULT.
-  PENDING_DEFAULT means the payment method is awaiting activation and will become default once activated.
+  The payment method status - ACTIVE, PENDING.
+  If ACTIVE, the payment method is ready to use for payins or payouts depending on if it's a payin or payout method.
+  If PENDING, the payment method has been set up but is still awaiting activation and cannot be used for payins or
+  payouts until then. Once activated, the status will change to ACTIVE.
   """
   status: MemberPaymentMethodStatus!
   """
-  True if this is the default payment method. Only one ACTIVE payment method can be default at a time.
-  If status is PENDING then payment method will become the default once activated.
+  This is 'true' for only one of the members ACTIVE methods which is the default payment method that will be used for
+  charging or payout the member. For PENDING methods, this can also be 'true' if the member has chosen to set up this
+  payment method as default during the setup process.
   """
   isDefault: Boolean!
   """
-  Specific details of the actual connection - e.g. a bank account reference, phone number for swish,
+  Specific details of the actual connection if method is ACTIVE - e.g. a bank account reference, phone number for swish,
   or email/kivra for invoice.
   """
   details: PaymentMethodDetails!
 }
 type MemberPaymentMethods {
   """
-   List of active and pending payment payin methods for this member. 
+  List of all member's ACTIVE and PENDING payment payin methods.
+  A member can have multiple ACTIVE payment methods with these constraints:
+  - Only one ACTIVE payment method per provider, eg. one Trustly, one Swish, one Nordea etc.
+  - Only one ACTIVE payment method can be default at a time.
+  A member can have multiple PENDING payment methods with these constraints:
+  - Only one PENDING payment method per provider, eg. one Trustly, one Swish, one Nordea etc.
+  So there can exist max two payment methods per provider, one ACTIVE and one PENDING.
+  If a PENDING payment method has isDefault=true, then it will become the default ACTIVE payment method once activated.
   """
   payinMethods: [MemberPaymentMethod!]!
   """
-   List of active and pending payment payout methods for this member. 
+  List of all member's ACTIVE and PENDING payment payout methods.
+  A member can have multiple ACTIVE payment methods with these constraints:
+  - Only one ACTIVE payment method per provider, eg. one Trustly, one Swish, one Nordea etc.
+  - Only one ACTIVE payment method can be default at a time.
+  A member can have multiple PENDING payment methods with these constraints:
+  - Only one PENDING payment method per provider, eg. one Trustly, one Swish, one Nordea etc.
+  So there can exist max two payment methods per provider, one ACTIVE and one PENDING.
+  If a PENDING payment method has isDefault=true, then it will become the default ACTIVE payment method once activated.
   """
   payoutMethods: [MemberPaymentMethod!]!
   """
-   The default payment method for payin if any. 
+  The default payment method to use for payins if any.
+  Note that there can exist a PENDING payment method in `payinMethods` list with `isDefault`=true, in that case this default
+  payment method will be replaced by it once the pending method is activated.
   """
   defaultPayinMethod: MemberPaymentMethod
   """
-   The default payment method for payout if any. 
+  The default payment method to use for payouts if any.
+  Note that there can exist a PENDING payment method in `payoutMethods` list with `isDefault`=true, in that case this default
+  payment method will be replaced by it once the pending method is activated.
   """
   defaultPayoutMethod: MemberPaymentMethod
   """
-   The available payment methods that the member can choose from when setting up a new payment method. 
+  The available payment methods that the member can choose from when setting up a new payment method.
+  This list can include both payment methods that the member has already set up and new payment methods that the
+  member has not yet set up but are available to them. For already set up payment methods, the `isActive` field will
+  be true and the `details` field will be populated with the payment method details. For new payment methods that the
+  member has not yet set up, the `isActive` field will be false and the `details` field will be null.
+  If member picks a new payment method to set up, the corresponding mutation for setting up that payment method should
+  be called, eg. `paymentMethodSetupTrustly`, `paymentMethodSetupSwishPayin` etc.
+  If member picks an already set up payment method to set up as default, then `paymentMethodSetDefaultPayin` or
+  `paymentMethodSetDefaultPayout` mutation should be called depending on if it's a payin or payout method.
   """
   availableMethods: [MemberPaymentAvailablePaymentMethod!]!
   """
@@ -3392,9 +3440,9 @@ type Mutation {
   """
    Setup invoice payment method for the member. Kivra will be used as the provider if supported, else mail. 
   """
-  paymentMethodSetupInvoicePayin(input: PaymentMethodSetupInvoicePayinInput!): PaymentMethodSetupOutput!
+  paymentMethodSetupInvoicePayin: PaymentMethodSetupOutput!
   """
-   Setup Trustly payment payin and payout method for the member. 
+   Setup Trustly payment payin and payout method for the member. Requires member consent via redirect to Trustly URL in response.
   """
   paymentMethodSetupTrustly(input: PaymentMethodSetupTrustlyInput!): PaymentMethodSetupOutput!
   """
@@ -3406,17 +3454,19 @@ type Mutation {
   """
   paymentMethodSetupSwishPayout(input: PaymentMethodSetupSwishInput!): PaymentMethodSetupOutput!
   """
-   Setup Swish payin method for the member. 
+   Setup Swish payin method for the member. Requires member consent in Swish app. 
   """
   paymentMethodSetupSwishPayin(input: PaymentMethodSetupSwishInput!): PaymentMethodSetupOutput!
   """
-   Revoke an active payment method. The member will be required to set up a new payment method if they revoke their default one. 
+  A member can have multiple ACTIVE payment methods where one of those is default. This mutation changes the
+  members default payment method for charging to any of his/hers other active payment methods.
   """
-  paymentMethodRevoke(id: ID!): UserError
+  paymentMethodSetDefaultPayin(provider: MemberPaymentProvider!): UserError
   """
-   Set an active payment method as default. 
+  A member can have multiple ACTIVE payment methods where one of those is default. This mutation changes the
+  members default payment method for payouts to any of his/hers other active payment methods.
   """
-  paymentMethodSetDefault(id: ID!): UserError
+  paymentMethodSetDefaultPayout(provider: MemberPaymentProvider!): UserError
   """
   Start a conversation. This is effectively creating one, but with two slight differences from a regular
   "create something"-mutation:
@@ -3707,17 +3757,7 @@ type PaymentMethodInvoiceDetails {
   """
   email: String
 }
-input PaymentMethodSetupInvoicePayinInput {
-  """
-   Set up invoice payment method as default. 
-  """
-  setAsDefaultPayout: Boolean!
-}
 input PaymentMethodSetupNordeaPayoutInput {
-  """
-   Set up Nordea payout method as default. 
-  """
-  setAsDefault: Boolean!
   """
    The clearing number for member's bank account. 
   """
@@ -3756,24 +3796,12 @@ enum PaymentMethodSetupStatus {
   FAILED
 }
 input PaymentMethodSetupSwishInput {
-  """
-   Set up Swish payment method as default. 
-  """
-  setAsDefault: Boolean!
   """
    The Swish mobile number to use for payout or payin. 
   """
   phoneNumber: String!
 }
 input PaymentMethodSetupTrustlyInput {
-  """
-   Set up Trustly payment method as default for payin. 
-  """
-  setAsDefaultPayin: Boolean!
-  """
-   Set up Trustly payment method as default for payout. 
-  """
-  setAsDefaultPayout: Boolean!
   """
    The URL to redirect the member back to after a successful setup after Trustly onboarding. 
   """
@@ -3905,6 +3933,11 @@ type PriceIntent {
   When 'true' it means user has gone trough Insurely flow with that price intent
   """
   hasCollectedInsurelyData: Boolean!
+  """
+  List of external insurances fetched via Insurely that correspond to products Hedvig offers.
+  Null when no Insurely data collection has been associated with this price intent.
+  """
+  fetchedExternalInsurances: [FetchedExternalInsurance!]
 }
 enum PriceIntentAnimal {
   CAT

app/app/build.gradle.kts

@@ -195,8 +195,7 @@ dependencies {
   implementation(projects.featureCrossSellSheet)
   implementation(projects.featureDeleteAccount)
   implementation(projects.featureEditCoinsured)
-  implementation(projects.featureFlagsAndroid)
-  implementation(projects.featureFlagsPublic)
+  implementation(projects.featureFlags)
   implementation(projects.featureForever)
   implementation(projects.featureHelpCenter)
   implementation(projects.featureHome)

app/app/src/main/kotlin/com/hedvig/android/app/di/ApplicationModule.kt

@@ -358,7 +358,7 @@ val applicationModule = module {
   )
 }
 
-private class AndroidBuildConfig() : AppBuildConfig {
+private class AndroidBuildConfig : AppBuildConfig {
   override val debug: Boolean = BuildConfig.DEBUG
   override val applicationId: String = BuildConfig.APPLICATION_ID
   override val buildType: String = BuildConfig.BUILD_TYPE

app/app/src/main/kotlin/com/hedvig/android/app/navigation/HedvigNavHost.kt

@@ -440,6 +440,7 @@ internal fun HedvigNavHost(
     helpCenterGraph(
       hedvigDeepLinkContainer = hedvigDeepLinkContainer,
       navController = navController,
+      onNavigateUp = navController::navigateUp,
       onNavigateToQuickLink = onNavigateToQuickLink@{ quickLinkDestination ->
         val destination: Destination = when (quickLinkDestination) {
           QuickLinkChangeAddress -> {

app/core/core-datastore-public/build.gradle.kts

@@ -2,6 +2,7 @@ plugins {
   id("hedvig.multiplatform.library")
   id("hedvig.multiplatform.library.android")
   id("hedvig.gradle.plugin")
+  alias(libs.plugins.kmpNativeCoroutines)
 }
 
 kotlin {

app/core/core-resources/build.gradle.kts

@@ -33,7 +33,6 @@ lokalise {
 kotlin {
   androidLibrary {
     namespace = "hedvig.resources"
-    @Suppress("UnstableApiUsage")
     androidResources.enable = true
   }
   sourceSets {

app/data/data-addons/build.gradle.kts

@@ -22,7 +22,7 @@ dependencies {
   implementation(projects.apolloOctopusPublic)
   implementation(projects.coreCommonPublic)
   implementation(projects.coreDemoMode)
-  implementation(projects.featureFlagsPublic)
+  implementation(projects.featureFlags)
 
   testImplementation(libs.apollo.testingSupport)
   testImplementation(libs.assertK)

app/data/data-changetier/build.gradle.kts

@@ -25,7 +25,7 @@ dependencies {
   implementation(projects.dataContract)
   implementation(projects.dataCrossSellAfterFlow)
   implementation(projects.dataProductVariantPublic)
-  implementation(projects.featureFlagsPublic)
+  implementation(projects.featureFlags)
   implementation(projects.uiTiersAndAddons)
 
   testImplementation(libs.apollo.testingSupport)

app/data/data-conversations/build.gradle.kts

@@ -1,16 +1,21 @@
 plugins {
-  id("hedvig.jvm.library")
+  id("hedvig.multiplatform.library")
   id("hedvig.gradle.plugin")
 }
 
 hedvig {
   apollo("octopus")
 }
 
-dependencies {
-  implementation(libs.apollo.normalizedCache)
-  implementation(libs.arrow.core)
-  implementation(libs.koin.core)
-  implementation(projects.apolloCore)
-  implementation(projects.apolloOctopusPublic)
+kotlin {
+  sourceSets {
+    commonMain.dependencies {
+      implementation(libs.apollo.normalizedCache)
+      implementation(libs.arrow.core)
+      implementation(libs.koin.core)
+      implementation(projects.apolloCore)
+      implementation(projects.apolloOctopusPublic)
+      implementation(projects.loggingPublic)
+    }
+  }
 }

app/data/data-termination/build.gradle.kts

@@ -1,22 +1,26 @@
 plugins {
-  id("hedvig.jvm.library")
+  id("hedvig.multiplatform.library")
   id("hedvig.gradle.plugin")
 }
 
 hedvig {
   apollo("octopus")
 }
 
-dependencies {
-  implementation(libs.apollo.normalizedCache)
-  implementation(libs.apollo.runtime)
-  implementation(libs.arrow.core)
-  implementation(libs.koin.core)
-  implementation(libs.kotlinx.datetime)
-  implementation(projects.apolloCore)
-  implementation(projects.apolloOctopusPublic)
-  implementation(projects.coreCommonPublic)
-  implementation(projects.coreUiData)
-  implementation(projects.dataContract)
-  implementation(projects.featureFlagsPublic)
+kotlin {
+  sourceSets {
+    commonMain.dependencies {
+      implementation(libs.apollo.normalizedCache)
+      implementation(libs.apollo.runtime)
+      implementation(libs.arrow.core)
+      implementation(libs.koin.core)
+      implementation(libs.kotlinx.datetime)
+      implementation(projects.apolloCore)
+      implementation(projects.apolloOctopusPublic)
+      implementation(projects.coreCommonPublic)
+      implementation(projects.coreUiData)
+      implementation(projects.dataContract)
+      implementation(projects.featureFlags)
+    }
+  }
 }

app/datadog/datadog-core/build.gradle.kts

@@ -1,8 +1,12 @@
 plugins {
-  id("hedvig.jvm.library")
+  id("hedvig.multiplatform.library")
   id("hedvig.gradle.plugin")
 }
 
-dependencies {
-  implementation(libs.coroutines.core)
+kotlin {
+  sourceSets {
+    commonMain.dependencies {
+      implementation(libs.coroutines.core)
+    }
+  }
 }

app/design-system/design-system-hedvig/src/commonMain/kotlin/com/hedvig/android/design/system/hedvig/InteractiveComponentSize.kt

@@ -16,6 +16,7 @@ import androidx.compose.ui.unit.Constraints
 import androidx.compose.ui.unit.Dp
 import androidx.compose.ui.unit.dp
 import androidx.compose.ui.unit.isSpecified
+import com.hedvig.android.design.system.hedvig.internal.identityHashCode
 import kotlin.math.roundToInt
 
 /**
@@ -36,7 +37,7 @@ internal object MinimumInteractiveModifier : ModifierNodeElement<MinimumInteract
       "interactions if the element would measure smaller"
   }
 
-  override fun hashCode(): Int = System.identityHashCode(this)
+  override fun hashCode(): Int = identityHashCode(this)
 
   override fun equals(other: Any?) = (other === this)
 }