Fix username sent to IRC : prefer display_name over user.name

[mmarchand]

Try to use display_name (often set to the user's nick) over user.name when it is set

[ekmartin]

The fact that display names can be changed as often as you want, and that they're not unique, makes me slightly nervous. You could change your display name to someone else's username, post a message, and change back.

Or am I missing something here?

[mmarchand]

just like on IRC actually ? ;)

I have had about 100 users that expected to see their display names rather than the username (they are migrating from IRC to a public slack channel) and they could not change their username back to their IRC nickname after creating their account

[xrchz]

I think you may need to augment test/stubs/data-store-stub.js a bit to get the checks to pass.

[Stoner19]

I seem to be having some similar issues. A Slack user's real name is displayed on IRC when it should be their display name, and if there is a word/string said in IRC that matches something within their registration email (which is hidden from public) it changes that word on slack to their user name.