Conversation
title => html_enc($message), | ||
subtitle => 'Email address validation: '.html_enc($address) | ||
title => $message, | ||
subtitle => 'Email address validation: '.$address |
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.
Want to switch this to Email address validation: $address"
while you're at it?
And any others now enabled too 😄
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.
@GuiltyDolphin Sure 😄 Thanks for pointing it out!
@GuiltyDolphin The build stopped midway. Could you please look into it? 😄 |
@manrajgrover Still an issue? Build seems to have finished from what I can tell. |
@GuiltyDolphin Nope, looks like someone else restarted it. |
@GuiltyDolphin I guess we can merge this |
@@ -43,7 +43,7 @@ handle remainder => sub { | |||
return unless $info; | |||
my (@vendor) = split(/\\n/, $info, 2); | |||
|
|||
my ($name, $addr) = map { html_enc($_); } @vendor; | |||
my ($name, $addr) = map { $_; } @vendor; |
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.
A.k.a @vendor
(mapping the identity is the same as the list).
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.
@GuiltyDolphin Fixed :)
@manrajgrover Thanks! |
What does your Pull Request do (check all that apply)?
Choose the most relevant items and use the following title template to name
your Pull Request.
New {Cheat Sheet Name} Cheat Sheet
New {IA Name} Instant Answer
{More than 1}: Fix {Removes html_enc}
{IA Name}: {Description of Improvements}
{GoodieRole/Templates/Tests/Docs}: {Short Description}
Description of changes
Remove
html_enc
from few IA GoodiesWhich issues (if any) does this fix?
#3563
People to notify (@mention interested parties)
@moollaza
Instant Answer Page:
Maintainer: @mention
None