Closed
Bug 1232978
Opened 10 years ago
Closed 9 years ago
Telegram URLs have an extra slash ("/") at the end causing non-working links
Categories
(Participation Infrastructure :: Phonebook, defect)
Participation Infrastructure
Phonebook
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: asdofindia, Assigned: shine, Mentored)
Details
What I did:
* Added "akshay" to Telegram in profile http://mozillians.org/u/asdofindia
* Clicked on it to reach https://telegram.me/akshay/
What happened:
Telegram is confused on what to do with that URL
What should have happened:
The URL should have been https://telegram.me/akshay
Without the trailing slash, Telegram happily opens a chat with me.
Probable reason:
This might have been introduced by a recent change in URL pattern from Telegram's side. They now support telegram.me/share/url and lot such patterns.
Possible fix:
Change https://github.com/mozilla/mozillians/blob/1e5b46ef47eeb4da977ea38f8da0a5b88a0db94d/mozillians/users/models.py#L842 line to remove the trailing slash
Assignee | ||
Comment 1•10 years ago
|
||
I can take this up to fix it.
Comment 2•10 years ago
|
||
Hey :shine!
Thanks for the contribution. Are you familiar with our dev environment? Feel free to reach out to #commtools on irc.mozilla.org if you have any further question.
Do you want me to assign this bug to you?
Flags: needinfo?(radioactive.life)
Assignee | ||
Comment 3•10 years ago
|
||
Hey :nemo,
Thanks to Akshay, I got the dev environment working last night. I've hit a few bottlenecks in between (that probably might be because I'm new to the environment). I'll reach out to you guys if I need some help.
Yes, you can assign this bug to me. I'll fix this bug soon.
Flags: needinfo?(radioactive.life)
Updated•10 years ago
|
Assignee: nobody → radioactive.life
Updated•10 years ago
|
Version: 2015-12.6 → other
Assignee | ||
Comment 4•9 years ago
|
||
Pull request https://github.com/mozilla/mozillians/pull/1287 was merged on Jan 11, 2016
Assignee | ||
Comment 5•9 years ago
|
||
:nemo, can you mark this bug as RESOLVED-FIXED, since I don't have the necessary permissions to resolve this bug as FIXED.
Flags: needinfo?(jgiannelos)
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Flags: needinfo?(jgiannelos)
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•