New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MBS-7307: Show LinkedIn URLs in the sidebar #424
MBS-7307: Show LinkedIn URLs in the sidebar #424
Conversation
|
LinkedIn. |
1f816a6
to
c6b7ab6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! 👍
c6b7ab6
to
86e1e79
Compare
86e1e79
to
f45d006
Compare
@y-van-z |
This is something that should be done in URL cleanup on input. The display code should only deal with cleaned up URLs. |
@uklauer I was talking about the subdomain in the link, ca.linkedin.com |
I think the subdomain should be removed entirely in URL cleanup, since it's not necessary to view the profile (it appears that you can view any profile from any country's subdomain). As of this writing, there are 305 LinkedIn URLs with two-letter subdomains. |
Created MBS-9188 |
No description provided.