accalia
(Apparently Archetype)
1
A slash preceeding an @mention causes the markup for the user card link to not be generated.
Example:
@codinghorror/@sam
this should have generated two notifications, but only contain the usercard link for Jeff.
EDIT… of course it doesn’t do it here. works on what.thedailywtf.com as well as try.d (link while it lasts)
EDIT2: nope. i just used the wrong slash.
10 Likes
accalia
(Apparently Archetype)
2
looks like quite a few chaacters also cause the bug:
http://what.thedailywtf.com/t/does-it-have-to-be-a-slash/8719/3
for example these three constructs generated notifications, but are missing mention hyperlink in cooked:
this.looks.like.an.email.but.isnt@accalia.fakedomain
x@accalia
_@accalia
this.looks.like.an.email.but.isnt@accalia.fakedomain
x@accalia
_@accalia
1 Like
sam
(Sam Saffron)
3
@eviltrout I think this is cause of
https://github.com/discourse/discourse/commit/63f2187d72356dedc26d5e1d727580f2ba87b132
But there may be other spots in the mention matcher as well that are causing this.
I’ve put in a pull request that resolves the mention-after-a-slash case.
4 Likes