Chat message hooks - external-user-id shows user name rather than ID (and is missing in 3.0)

[MWWorks]

In messageMapping.js:

chatTemplate(messageObj)
...
          "sender":{
            "internal-user-id": body.msg.sender.id,
            "external-user-id": body.msg.sender.name,
            "timezone-offset": body.msg.fromTimezoneOffset,
            "time": body.msg.timestamp
          }

external-user-id actually returns the external user name - also this seems missing completely in v3's event.js. The external user ID makes this more useful on the receiving server end.

This seems transitional between 2.6 and 3 so I'm not sure if I should add a PR - but I have fixed mine with:

chatTemplate(messageObj)
...
          "sender":{
            "internal-user-id": body.msg.sender.id,
            "external-user-id": UserMapping.getExternalUserID(body.msg.sender.id), //mw fix this to id instead of name above
            "user-name": body.msg.sender.name, //and provide name separately
            "timezone-offset": body.msg.fromTimezoneOffset,
            "time": body.msg.timestamp
          }

[prlanzarin]

Thanks for the report, will review in v2 and v3.