[IMPROVE] Filter markdown in notifications #9995
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
c0dzilla
force-pushed
the
markdown-filter
branch
from
6f0822b to
43ffb4d
Compare
c0dzilla
force-pushed
the
markdown-filter
branch
6 times, most recently
from
1503b79 to
b9601ba
Compare
c0dzilla
force-pushed
the
markdown-filter
branch
3 times, most recently
from
37fc644 to
d5ef312
Compare
c0dzilla
force-pushed
the
markdown-filter
branch
from
d5ef312 to
9b82798
Compare
|
@c0dzilla can you check the unit tests? |
c0dzilla
force-pushed
the
markdown-filter
branch
from
cd0f7ae to
7ccec1f
Compare
c0dzilla
force-pushed
the
markdown-filter
branch
from
7ccec1f to
456f0ee
Compare
|
@rodrigok fixed the tests. |
|
|
rodrigok
requested changes
Apr 20, 2020
rodrigok
approved these changes
Apr 20, 2020
rodrigok
requested changes
Apr 21, 2020
rodrigok
approved these changes
Apr 21, 2020
gabriellsh
added a commit
that referenced
this pull request
Apr 22, 2020
…users_and_rooms * 'develop' of github.com:RocketChat/Rocket.Chat: (29 commits) [FIX] 2FA not showing codes for Spanish translation (#17378) [NEW] [ENTERPRISE] Restrict the permissions configuration for guest users (#17333) [NEW] Federation event for when users left rooms (#17091) [FIX] CSV Importer fails when there are no users to import (#16790) Import slack's mpims as direct rooms instead of private groups (#17206) [FIX] SAML Idp Initiated Logout Error (#17324) [NEW] Better Push and Email Notification logic (#17357) [NEW] Error page when browser is not supported (#17372) [NEW] [ENTERPRISE] Omnichannel queue priorities (#17141) [IMPROVE] Change the SAML metadata order to conform to XSD specification (#15488) [IMPROVE] Filter markdown in notifications (#9995) [IMPROVE] User gets UI feedback when message is pinned or unpinned (#16056) Remove set as alias setting (#16343) [IMPROVE] Add `file-title` and `file-desc` as new filter tag options on message search (#16858) [NEW] Add ability to set tags in the Omnichannel room closing dialog (#17254) [FIX] Show active admin and user account menu item (#17047) [NEW] [ENTERPRISE] Allows to set a group of departments accepted for forwarding chats (#17335) [FIX] Prevent user from getting stuck on login, if there is some bad fname (#17331) [FIX] Remove properties from users.info response (#17238) Bump version to 3.1.1 ...
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@RocketChat/core
Closes #9979
This pr adds a filter to remove markdown tags from desktop/push notifications.
Example:
gives notification
