Skip to content

[IMPROVE] Filter markdown in notifications#9995

Merged
rodrigok merged 9 commits intoRocketChat:developfrom
c0dzilla:markdown-filter
Apr 21, 2020
Merged

[IMPROVE] Filter markdown in notifications#9995
rodrigok merged 9 commits intoRocketChat:developfrom
c0dzilla:markdown-filter

Conversation

@c0dzilla
Copy link
Copy Markdown
Contributor

@c0dzilla c0dzilla commented Mar 3, 2018

@RocketChat/core

Closes #9979

This pr adds a filter to remove markdown tags from desktop/push notifications.

Example:

testingfilter

gives notification
filter_notification

@RocketChat RocketChat deleted a comment Mar 3, 2018
@RocketChat RocketChat deleted a comment Mar 3, 2018
@RocketChat RocketChat deleted a comment Mar 3, 2018
@RocketChat RocketChat deleted a comment Mar 3, 2018
@RocketChat RocketChat deleted a comment Mar 3, 2018
@c0dzilla c0dzilla force-pushed the markdown-filter branch 6 times, most recently from 1503b79 to b9601ba Compare March 5, 2018 19:11
@RocketChat RocketChat deleted a comment Mar 5, 2018
@RocketChat RocketChat deleted a comment Mar 5, 2018
@RocketChat RocketChat deleted a comment Mar 5, 2018
@RocketChat RocketChat deleted a comment Mar 5, 2018
@RocketChat RocketChat deleted a comment Mar 5, 2018
@RocketChat RocketChat deleted a comment Mar 5, 2018
@RocketChat RocketChat deleted a comment Mar 5, 2018
@RocketChat RocketChat deleted a comment Mar 5, 2018
@RocketChat RocketChat deleted a comment Mar 5, 2018
@RocketChat RocketChat deleted a comment Mar 5, 2018
@RocketChat RocketChat deleted a comment Mar 5, 2018
@RocketChat RocketChat deleted a comment Mar 5, 2018
@RocketChat RocketChat deleted a comment Mar 5, 2018
@RocketChat RocketChat deleted a comment Mar 5, 2018
@RocketChat RocketChat deleted a comment Mar 5, 2018
@RocketChat RocketChat deleted a comment Mar 5, 2018
@RocketChat RocketChat deleted a comment Mar 5, 2018
@RocketChat RocketChat deleted a comment Mar 5, 2018
@sampaiodiego sampaiodiego modified the milestones: 1.3.0, 1.4.0 Jul 25, 2019
@sampaiodiego sampaiodiego modified the milestones: 2.0.0, 3.0.0 Sep 6, 2019
@rodrigok rodrigok modified the milestones: 2.4.0, 3.0.0 Dec 19, 2019
@c0dzilla c0dzilla force-pushed the markdown-filter branch 3 times, most recently from 37fc644 to d5ef312 Compare February 26, 2020 18:23
@rodrigok
Copy link
Copy Markdown
Member

@c0dzilla can you check the unit tests?

@c0dzilla
Copy link
Copy Markdown
Contributor Author

c0dzilla commented Mar 2, 2020

@rodrigok fixed the tests.

@engelgabriel engelgabriel removed this from the 3.0.0 milestone Mar 17, 2020
@CLAassistant
Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@sampaiodiego sampaiodiego mentioned this pull request Apr 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[IMPROVEMENT] Remove all markdown from Push Notification

8 participants