-
Notifications
You must be signed in to change notification settings - Fork 2.7k
[MM-20480] Migrate new_message_separator and its tests to typescript #6636
[MM-20480] Migrate new_message_separator and its tests to typescript #6636
Conversation
Hello @abdusabri, Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here. |
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.
LGTM! Thanks @abdusabri!
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.
LGTM
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.
Thanks @abdusabri - tested around new messages separator and looks good to me 👍
@sudheerDev - please resolve associated Jira ticket after merge - thanks!
Test server destroyed |
/update-branch |
Thanks for yet another contribution @abdusabri. Glad you are continuously contributing 🚀 |
Thanks Ben! Happy to help 😊 |
Summary
Migrates
new_message_separator
module and its tests to TypeScript.Ticket Link
Fixes mattermost/mattermost#15461, https://mattermost.atlassian.net/browse/MM-20480