diff options
author | Harrison Healey <harrisonmhealey@gmail.com> | 2016-03-31 12:21:20 -0400 |
---|---|---|
committer | Harrison Healey <harrisonmhealey@gmail.com> | 2016-03-31 12:21:20 -0400 |
commit | 8050a9b958f72227dbf601c05f6bcdfe50141708 (patch) | |
tree | 4e3df4d2d0e09b1bbcdd5f11207e61368c0c07bf /webapp/components/posts_view.jsx | |
parent | 8abd74b89222fb09a5bb3783037f27eeb86303fc (diff) | |
parent | 980419b37cb1a329cc694213b2648b5919e4902c (diff) | |
download | chat-8050a9b958f72227dbf601c05f6bcdfe50141708.tar.gz chat-8050a9b958f72227dbf601c05f6bcdfe50141708.tar.bz2 chat-8050a9b958f72227dbf601c05f6bcdfe50141708.zip |
Merge pull request #2589 from mattermost/plt-1812
PLT-1812 Update webhook posts to never show in collapsed view
Diffstat (limited to 'webapp/components/posts_view.jsx')
-rw-r--r-- | webapp/components/posts_view.jsx | 21 |
1 files changed, 5 insertions, 16 deletions
diff --git a/webapp/components/posts_view.jsx b/webapp/components/posts_view.jsx index 647c7f086..af849fc4a 100644 --- a/webapp/components/posts_view.jsx +++ b/webapp/components/posts_view.jsx @@ -173,24 +173,15 @@ export default class PostsView extends React.Component { const postFromWebhook = Boolean(post.props && post.props.from_webhook); const prevPostFromWebhook = Boolean(prevPost.props && prevPost.props.from_webhook); const prevPostUserId = Utils.isSystemMessage(prevPost) ? '' : prevPost.user_id; - let prevWebhookName = ''; - if (prevPost.props && prevPost.props.override_username) { - prevWebhookName = prevPost.props.override_username; - } - let curWebhookName = ''; - if (post.props && post.props.override_username) { - curWebhookName = post.props.override_username; - } // consider posts from the same user if: // the previous post was made by the same user as the current post, // the previous post was made within 5 minutes of the current post, - // the previous post and current post are both from webhooks or both not, - // the previous post and current post have the same webhook usernames + // the current post is not from a webhook + // the previous post is not from a webhook if (prevPostUserId === postUserId && post.create_at - prevPost.create_at <= 1000 * 60 * 5 && - postFromWebhook === prevPostFromWebhook && - prevWebhookName === curWebhookName) { + !postFromWebhook && !prevPostFromWebhook) { sameUser = true; } @@ -213,13 +204,11 @@ export default class PostsView extends React.Component { // the previous post was made by the same user as the current post, // the previous post is not a comment, // the current post is not a comment, - // the previous post and current post are both from webhooks or both not, - // the previous post and current post have the same webhook usernames + // the current post is not from a webhook if (prevPostUserId === postUserId && !prevPostIsComment && !postIsComment && - postFromWebhook === prevPostFromWebhook && - prevWebhookName === curWebhookName) { + !postFromWebhook) { hideProfilePic = true; } } |