Closed Bug 1191720 Opened 6 years ago Closed 6 years ago

Add Telegram to the IM List

Categories

(Participation Infrastructure :: Phonebook, defect)

2015-8.2
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: david.weir, Assigned: david.weir)

Details

I cant see this on master so I am going to add telegram as a lot of people are using it now
Please see https://github.com/mozilla/mozillians/compare/master...satdav:patch-3
Assignee: nobody → david.weir
Status: NEW → ASSIGNED
Hey :satdav

Thanks for your PR. Code needs some changes and will comment inline at the PR. First we need to make sure that we need that functionality.

@pierros
What do you think?
Flags: needinfo?(pierros)
Nemo hey the reason I added this was because a lot of Mozilla is using it ps I can tell you there is over 200 by the looks as there is a Mozillians one and it's always full ps I changed a bit more to the code and jenkins has checked it and said it looks fine for code wise
This makes sense as an addition. Thanks for your commit satdav!

Nemo please proceed with r.
Flags: needinfo?(pierros)
https://github.com/mozilla/mozillians/pull/1221

Thanks to nikos for merging my 2 together
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Commits pushed to master at https://github.com/mozilla/mozillians

https://github.com/mozilla/mozillians/commit/b34c6dff71fe1dc4cb2a7d619918207af8591d89
[Fix bug 1191720] Add Telegram to Social Media Section

https://github.com/mozilla/mozillians/commit/eea616b8d031f3ab26c87d51205d0540ce21d012
Merge pull request #1221 from comzeradd/telegram

[Fix bug 1191720] Add Telegram to Social Media Section
Version: other → next
Verified on stage:

* Edit profile has "External Accounts" entry for "Telegram"
* Added telegram entry to my profile successfully
* Link rendered to my profile page redirected to a telegram chat for the specified user.
Status: RESOLVED → VERIFIED
Version: next → 2015-8.2
You need to log in before you can comment on or make changes to this bug.