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] "Highlight Words" wasn't working with more than one word #10083

Merged
merged 3 commits into from
Apr 17, 2018

Conversation

nemani
Copy link
Contributor

@nemani nemani commented Mar 9, 2018

@RocketChat/core

Closes #10077

Fixed typo `Hilights` to `Highlights`
We were splitting the text only by newlines, this commit splits the text by both newlines and commas
While joing again, we will join by both newline and comma.

Fixes RocketChat#10077
@CLAassistant
Copy link

CLAassistant commented Mar 9, 2018

CLA assistant check
All committers have signed the CLA.

@theorenck theorenck requested a review from rodrigok March 9, 2018 22:18
@theorenck
Copy link
Contributor

Thanks @nemaniarjun. @rodrigok can you review this one?

@nemani
Copy link
Contributor Author

nemani commented Mar 26, 2018

PING! I would like to get this merged so I can write about it in my GSoC proposal!
Thanks :D

@gdelavald gdelavald self-assigned this Apr 12, 2018
Copy link
Contributor

@gdelavald gdelavald left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Working as intended.

@gdelavald gdelavald removed their assignment Apr 12, 2018
@rodrigok rodrigok changed the title [FIX] Fix highlight multi words [FIX] "Highlight Words" wasn't working with more than one word Apr 17, 2018
@rodrigok rodrigok added this to the 0.64.0 milestone Apr 17, 2018
@rodrigok rodrigok merged commit 88c8070 into RocketChat:develop Apr 17, 2018
@rodrigok rodrigok mentioned this pull request Apr 28, 2018
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.

6 participants