Add Telegram to the IM List

VERIFIED FIXED

Status

Participation Infrastructure
Phonebook
VERIFIED FIXED
2 years ago
2 years ago

People

(Reporter: David Weir (satdav), Assigned: David Weir (satdav))

Tracking

2015-8.2

Details

(Assignee)

Description

2 years ago
I cant see this on master so I am going to add telegram as a lot of people are using it now
(Assignee)

Comment 1

2 years ago
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)
(Assignee)

Comment 3

2 years ago
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)
(Assignee)

Comment 5

2 years ago
https://github.com/mozilla/mozillians/pull/1221

Thanks to nikos for merging my 2 together
(Assignee)

Updated

2 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED

Comment 6

2 years ago
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

Updated

2 years ago
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.