Dependency segregation

nextmcloud · Jul 18, 2024 · 2eb3f37 · 2eb3f37
1 parent 03a72cb
commit 2eb3f37
Show file tree

Hide file tree

Showing 4 changed files with 35 additions and 15 deletions.
diff --git a/app/build.gradle b/app/build.gradle
@@ -53,7 +53,8 @@ apply plugin: 'com.google.devtools.ksp'
 
 
 println "Gradle uses Java ${Jvm.current()}"
-
+// apply MoEngage SDK for NMC
+apply from: "$rootProject.projectDir/nmc_moengage-dependencies.gradle"
 configurations {
     configureEach {
         exclude group: 'org.jetbrains', module: 'annotations-java5' // via prism4j, already using annotations explicitly
@@ -108,8 +109,6 @@ android {
         targetSdkVersion 34
         compileSdk 34
 
-        // NMC Customization
-        buildConfigField "String", "MOENGAGE_APP_ID", "${MOENGAGE_APP_ID}"
         buildConfigField 'boolean', 'CI', ciBuild.toString()
         buildConfigField 'boolean', 'RUNTIME_PERF_ANALYSIS', perfAnalysis.toString()
 
@@ -425,19 +424,11 @@ dependencies {
 
     implementation "io.coil-kt:coil:2.6.0"
 
-    // NMC: MoEngage Dependencies
-    // core moengage features
-    implementation(moengage.core)
-    // optionally add this to use the Push Templates feature
-    implementation(moengage.richNotification)
-    // optionally add this to use the InApp feature
-    implementation(moengage.inapp)
-    // Required for MoEngage to provide accurate analytics based on Advertising Identifier
-    implementation 'com.google.android.gms:play-services-ads-identifier:18.0.1'
-    // NMC: end
-
     // splash screen dependency ref: https://developer.android.com/develop/ui/views/launch/splash-screen/migrate
     implementation 'androidx.core:core-splashscreen:1.0.1'
+
+    // NMC: dependency required to capture Advertising ID for Adjust & MoEngage SDK
+    implementation 'com.google.android.gms:play-services-ads-identifier:18.0.1'
 }
 
 configurations.configureEach {

diff --git a/app/src/main/java/com/nmc/android/marketTracking/MoEngageSdkUtils.kt b/app/src/main/java/com/nmc/android/marketTracking/MoEngageSdkUtils.kt
@@ -95,7 +95,9 @@ object MoEngageSdkUtils {
             .configureNotificationMetaData(
                 NotificationConfig(
                     R.drawable.notification_icon,
-                    R.drawable.notification_icon
+                    R.drawable.notification_icon,
+                    R.color.primary,
+                    false
                 )
             )
             .build()

diff --git a/...src/main/java/com/owncloud/android/ui/dialog/ChooseRichDocumentsTemplateDialogFragment.kt b/...src/main/java/com/owncloud/android/ui/dialog/ChooseRichDocumentsTemplateDialogFragment.kt
@@ -28,6 +28,7 @@ import com.nextcloud.client.network.ClientFactory
 import com.nextcloud.client.network.ClientFactory.CreationException
 import com.nextcloud.utils.extensions.getParcelableArgument
 import com.owncloud.android.MainApp
+import com.nmc.android.marketTracking.MoEngageSdkUtils
 import com.owncloud.android.R
 import com.owncloud.android.databinding.ChooseTemplateBinding
 import com.owncloud.android.datamodel.FileDataStorageManager
@@ -384,6 +385,11 @@ class ChooseRichDocumentsTemplateDialogFragment :
                 putExtra(ExternalSiteWebView.EXTRA_TEMPLATE, template)
             }
 
+            // NMC: track create office file event
+            file?.let {
+                MoEngageSdkUtils.trackCreateFileEvent(MainApp.getAppContext(), it, template.type)
+            }
+
             fragment.run {
                 startActivity(intent)
                 dismiss()

diff --git a/nmc_moengage-dependencies.gradle b/nmc_moengage-dependencies.gradle
@@ -0,0 +1,21 @@
+/*
+ * Nextcloud - Android Client
+ *
+ * SPDX-FileCopyrightText: 2024 Your Name <[email protected]>
+ * SPDX-License-Identifier: AGPL-3.0-or-later
+ */
+
+android {
+    buildTypes.each {
+        it.buildConfigField "String", "MOENGAGE_APP_ID", "${MOENGAGE_APP_ID}"
+    }
+}
+
+dependencies {
+    // core moengage features
+    implementation(moengage.core)
+    // optionally add this to use the Push Templates feature
+    implementation(moengage.richNotification)
+    // optionally add this to use the InApp feature
+    implementation(moengage.inapp)
+}