applying the changes

modules/.submodules.json

@@ -19,7 +19,8 @@
     "idxIdSystem",
     "fabrickIdSystem",
     "verizonMediaIdSystem",
-    "pubProvidedIdSystem"
+    "pubProvidedIdSystem",
+    "mwOpenLinkIdSystem"
   ],
   "adpod": [
     "freeWheelAdserverVideo",

modules/mwOpenLinkIdSystem.js

@@ -114,7 +114,7 @@ function setID(configParams) {
 export { writeCookie };
 
 /** @type {Submodule} */
-export const mwOpenLinkSubModule = {
+export const mwOpenLinkIdSubModule = {
   /**
      * used to link submodule with config
      * @type {string}
@@ -127,10 +127,8 @@ export const mwOpenLinkSubModule = {
      * @return {(Object|undefined}
      */
   decode(mwOlId) {
-    if (mwOlId && utils.isPlainObject(mwOlId)) {
-      return { mwOlId };
-    }
-    return undefined;
+    const id = value ? utils.isStr(value) ? value : utils.isPlainObject(value) ? value.id : undefined : undefined;
+    return id ? { 'mwOpenLinkId': id } : undefined;
   },
 
   /**
@@ -142,9 +140,8 @@ export const mwOpenLinkSubModule = {
   getId(submoduleConfig) {
     const submoduleConfigParams = (submoduleConfig && submoduleConfig.params) || {};
     if (!isValidConfig(submoduleConfigParams)) return undefined;
-    const newId = setID(submoduleConfigParams);
-    return newId;
+    return setID(submoduleConfigParams);
   }
 };
 
-submodule('userId', mwOpenLinkSubModule);
+submodule('userId', mwOpenLinkIdSubModule);

modules/mwOpenLinkIdSystem.md

@@ -10,7 +10,7 @@ ex: $ gulp build --modules=userId,mwOpenLinkIdSystem
 
 ##$ MediaWallah openLink ID Example Configuration
 
-When the module is included, it's automatically enabled and saves an id to both cookie and local storage with an expiration time of 1 year.  
+When the module is included, it's automatically enabled and saves an id to both cookie with an expiration time of 1 year.  
 
 ### Prebid Params
 
@@ -21,9 +21,9 @@ pbjs.setConfig({
         userIds: [{
             name: 'mwOpenLinkId',
             params: {
-                accountId: 0000,
-                partnerId: 0000,
-                uid: '12345xyz'
+                accountId: '1000',
+                partnerId: '1001',
+                uid: 'u-123xyz'
             }
         }]
     }
@@ -34,7 +34,8 @@ The below parameters apply only to the MediaWallah OpenLink ID User ID Module in
 
 | Params under usersync.userIds[]| Scope | Type | Description | Example |
 | --- | --- | --- | --- | --- |
+| name | Required | String | The name of this module. | `'mwOpenLinkId'` |
 | params | Required | Object | Details for mwOLID syncing. | |
 | params.accountId | Required | String | The MediaWallah assigned Account Id | `1000` |
-| params.partnerId | Required | String | The MediaWallah assign partner Id | `1001` |
-| params.uid | Optional | String | Your unique Id for the user or browser. Used for matching| `u-123xyz` |
+| params.partnerId | Required | String | The MediaWallah assign Partner Id | `1001` |
+| params.uid | Optional | String | Your unique Id for the user or browser. Used for matching | `u-123xyz` |

modules/userId/eids.js

@@ -168,6 +168,12 @@ const USER_IDS_CONFIG = {
   'fabrickId': {
     source: 'neustar.biz',
     atype: 1
+  },
+
+  // MediaWallah OpenLink
+  'mwOpenLinkId': {
+    source: 'mediawallahscript.com',
+    atype: 1
   }
 };
 

modules/userId/eids.md

@@ -141,6 +141,14 @@ userIdAsEids = [
             id: 'some-random-id-value',
             atype: 1
         }]
+    },
+
+    {
+        source: 'mediawallahscript.com',
+        uids: [{
+            id: 'some-random-id-value',
+            atype: 1
+        }]
     }
 ]
 ```

modules/userId/userId.md

@@ -77,6 +77,13 @@ pbjs.setConfig({
                  name: '_criteoId',
                  expires: 1
               }
+        }, {
+            name: 'mwOpenLinkId',
+            params: {
+                accountId: 0000,
+                partnerId: 0000,
+                uid: '12345xyz'
+            }
         }],
         syncDelay: 5000,
         auctionDelay: 1000

test/spec/modules/mwOpenLinkIdSystem_spec.js

@@ -1,4 +1,4 @@
-import { writeCookie, mwOpenLinkSubModule } from 'modules/mwOpenLinkIdSystem.js';
+import { writeCookie, mwOpenLinkIdSubModule } from 'modules/mwOpenLinkIdSystem.js';
 
 const P_CONFIG_MOCK = {
   params: {
@@ -14,12 +14,12 @@ describe('mwOpenLinkId module', function () {
 
   it('getId() should return a MediaWallah openLink Id when the MediaWallah openLink first party cookie exists', function () {
     writeCookie({eid: 'XX-YY-ZZ-123'});
-    const id = mwOpenLinkSubModule.getId(P_CONFIG_MOCK);
+    const id = mwOpenLinkIdSubModule.getId(P_CONFIG_MOCK);
     expect(id).to.be.deep.equal({id: {eid: 'XX-YY-ZZ-123'}});
   });
 
   it('getId() should return an empty Id when the MediaWallah openLink first party cookie is missing', function () {
-    const id = mwOpenLinkSubModule.getId();
+    const id = mwOpenLinkIdSubModule.getId();
     expect(id).to.be.deep.equal(undefined);
   });
 });