From f013d91b89c03e49e66d0726cb66be886bc80c01 Mon Sep 17 00:00:00 2001 From: Joram Wilander Date: Thu, 1 Dec 2016 07:22:38 -0500 Subject: Count posts to DMs as mentions when from webhooks (#4676) --- api/post.go | 3 +++ webapp/components/backstage/backstage_controller.jsx | 4 ++-- webapp/components/backstage/components/backstage_navbar.jsx | 2 +- webapp/components/integrations/components/add_incoming_webhook.jsx | 2 +- webapp/components/integrations/components/confirm_integration.jsx | 2 +- .../integrations/components/installed_incoming_webhooks.jsx | 2 +- webapp/components/integrations/components/integrations.jsx | 2 +- 7 files changed, 10 insertions(+), 7 deletions(-) diff --git a/api/post.go b/api/post.go index c646b056a..ef9c0991d 100644 --- a/api/post.go +++ b/api/post.go @@ -600,6 +600,9 @@ func sendNotifications(c *Context, post *model.Post, team *model.Team, channel * } mentionedUserIds[otherUserId] = true + if post.Props["from_webhook"] == "true" { + mentionedUserIds[post.UserId] = true + } } else { keywords := getMentionKeywordsInChannel(profileMap) diff --git a/webapp/components/backstage/backstage_controller.jsx b/webapp/components/backstage/backstage_controller.jsx index 690880071..4890dfddb 100644 --- a/webapp/components/backstage/backstage_controller.jsx +++ b/webapp/components/backstage/backstage_controller.jsx @@ -14,7 +14,7 @@ export default class BackstageController extends React.Component { return { children: React.PropTypes.node.isRequired, params: React.PropTypes.object.isRequired, - user: React.PropTypes.user.isRequired + user: React.PropTypes.object.isRequired }; } @@ -68,4 +68,4 @@ export default class BackstageController extends React.Component { ); } -} \ No newline at end of file +} diff --git a/webapp/components/backstage/components/backstage_navbar.jsx b/webapp/components/backstage/components/backstage_navbar.jsx index 7bccfc9f7..d49c89b09 100644 --- a/webapp/components/backstage/components/backstage_navbar.jsx +++ b/webapp/components/backstage/components/backstage_navbar.jsx @@ -9,7 +9,7 @@ import {Link} from 'react-router/es6'; export default class BackstageNavbar extends React.Component { static get propTypes() { return { - team: React.propTypes.object.isRequired + team: React.PropTypes.object.isRequired }; } diff --git a/webapp/components/integrations/components/add_incoming_webhook.jsx b/webapp/components/integrations/components/add_incoming_webhook.jsx index fa8a47a9c..b65696965 100644 --- a/webapp/components/integrations/components/add_incoming_webhook.jsx +++ b/webapp/components/integrations/components/add_incoming_webhook.jsx @@ -15,7 +15,7 @@ import SpinnerButton from 'components/spinner_button.jsx'; export default class AddIncomingWebhook extends React.Component { static get propTypes() { return { - team: React.propTypes.object.isRequired + team: React.PropTypes.object.isRequired }; } diff --git a/webapp/components/integrations/components/confirm_integration.jsx b/webapp/components/integrations/components/confirm_integration.jsx index 1f043ca83..1cc1e727b 100644 --- a/webapp/components/integrations/components/confirm_integration.jsx +++ b/webapp/components/integrations/components/confirm_integration.jsx @@ -15,7 +15,7 @@ import Constants from 'utils/constants.jsx'; export default class ConfirmIntegration extends React.Component { static get propTypes() { return { - team: React.propTypes.object.isRequired, + team: React.PropTypes.object.isRequired, location: React.PropTypes.object, loading: React.PropTypes.bool }; diff --git a/webapp/components/integrations/components/installed_incoming_webhooks.jsx b/webapp/components/integrations/components/installed_incoming_webhooks.jsx index 243195b8b..0b6ea18c7 100644 --- a/webapp/components/integrations/components/installed_incoming_webhooks.jsx +++ b/webapp/components/integrations/components/installed_incoming_webhooks.jsx @@ -19,7 +19,7 @@ import {FormattedMessage} from 'react-intl'; export default class InstalledIncomingWebhooks extends React.Component { static get propTypes() { return { - team: React.propTypes.object.isRequired + team: React.PropTypes.object.isRequired }; } diff --git a/webapp/components/integrations/components/integrations.jsx b/webapp/components/integrations/components/integrations.jsx index ad780b605..9dc64d0cf 100644 --- a/webapp/components/integrations/components/integrations.jsx +++ b/webapp/components/integrations/components/integrations.jsx @@ -16,7 +16,7 @@ import * as Utils from 'utils/utils.jsx'; export default class Integrations extends React.Component { static get propTypes() { return { - team: React.propTypes.object.isRequired, + team: React.PropTypes.object.isRequired, user: React.PropTypes.object.isRequired }; } -- cgit v1.2.3-1-g7c22