Merge pull request #684 from herdwatch/fix-alternate-link-domain

fix duplicate alternate link domains

Author: echo-branch

src/scripts/__tests__/associatedDomains.test.js

@@ -0,0 +1,10 @@
+const iosEntitlements = require('../ios/updateAssociatedDomains');
+const androidManifest = require('../android/updateAndroidManifest');
+
+const preferences = {
+  iosLinkDomain: [],
+  androidLinkDomain: [],
+  linkDomain: ['cordova.app.link', 'cordova-alternate.app.link', 'test.app.link']
+}
+console.log(iosEntitlements.updateAssociatedDomains(preferences));
+console.log(androidManifest.getAppLinkIntentFilterData(preferences));

src/scripts/android/updateAndroidManifest.js

@@ -6,7 +6,8 @@
 
   // entry
   module.exports = {
-    writePreferences: writePreferences
+    writePreferences: writePreferences,
+    getAppLinkIntentFilterData: getAppLinkIntentFilterData
   };
 
   // injects config.xml preferences into AndroidManifest.xml file.
@@ -260,6 +261,10 @@
 
       // app.link link domains need -alternate associated domains as well (for Deep Views)
       if (linkDomain.indexOf("app.link") !== -1) {
+        const isAlternateDomain = linkDomain.indexOf("-alternate") !== -1;
+        if(isAlternateDomain){
+          continue;
+        }
         const first = linkDomain.split(".")[0];
         const rest = linkDomain
           .split(".")

src/scripts/ios/updateAssociatedDomains.js

@@ -12,7 +12,8 @@
 
   // entry
   module.exports = {
-    addAssociatedDomains: addAssociatedDomains
+    addAssociatedDomains: addAssociatedDomains,
+    updateAssociatedDomains: updateAssociatedDomains
   };
 
   // updates the associated domains from the link-domain field of the app's config.xml
@@ -127,6 +128,11 @@
     for (let i = 0; i < linkDomains.length; i++) {
       const linkDomain = linkDomains[i];
 
+      const isAlternateDomain = linkDomain.indexOf("-alternate") !== -1;
+      if(isAlternateDomain){
+        continue;
+      }
+
       // add link domain to associated domain
       domainList.push(prefix + linkDomain);