Skip to content
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

Fix #126 Replace <a> tags with Slack <link|name> #222

Merged
merged 1 commit into from
Nov 19, 2016

Conversation

frodeaa
Copy link
Contributor

@frodeaa frodeaa commented May 27, 2016

Replace <a href...>text</a> with <url|text> before
escaping the remaining parts of the message

fixes #126
fixes nishio-dens/bitbucket-pullrequest-builder-plugin#59

@frodeaa frodeaa changed the title Replace <a> tags with Slack <link|name> Fix #126 Replace <a> tags with Slack <link|name> May 27, 2016
@facastagnini
Copy link

@kmadel can this please be merged?

Thanks a lot

@mskutin
Copy link

mskutin commented Oct 21, 2016

@kmadel could you please take a look at this PR?

@frodeaa frodeaa force-pushed the a-tag-format-escape branch from 82d7103 to d2f8663 Compare October 21, 2016 18:31
@jgangemi
Copy link

jgangemi commented Nov 7, 2016

any chance this could be rebased and reviewed so it gets included in the 2.1 release? i'd really love to see this fixed.

@kmadel kmadel added this to the slack-2.2 milestone Nov 14, 2016
@kmadel
Copy link
Contributor

kmadel commented Nov 14, 2016

Once latest changes are merged from master, this is a candidate for 2.2

@frodeaa frodeaa force-pushed the a-tag-format-escape branch from d2f8663 to b9791af Compare November 16, 2016 12:46
Replace <a href...>text</a> with <url|text> before
escaping the remaining parts of the message

fixes jenkinsci#126
fixes nishio-dens/bitbucket-pullrequest-builder-plugin#59
@kmadel kmadel merged commit ca78846 into jenkinsci:master Nov 19, 2016
@samrocketman
Copy link
Member

Feel free to subscribe to #296 for updates related to Slack Plugin 2.2 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

HTML is not escaped properly Formatting/ escaping issue with Bitbucket Pull Request plugin
6 participants