mattermost-plugin-jira icon indicating copy to clipboard operation
mattermost-plugin-jira copied to clipboard

[GH-857]: Fixed Jira bot DMs you when you mention yourself in a comment

Open Kshitij-Katiyar opened this issue 2 years ago • 3 comments

Summary

  • Fixed the issue of Jira bot DMs you when you mention yourself in a comment

Issue Link

https://github.com/mattermost/mattermost-plugin-jira/issues/857

What to test

  • Jira bot sending you DM when you mention yourself in a comment

How to test

  • Install the plugin
  • Configure & setup the plugin
  • Create & connect the webhook
  • mention yourself in a comment on a Jira issue

Kshitij-Katiyar avatar Sep 20 '23 10:09 Kshitij-Katiyar

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 29.63%. Comparing base (d2183a1) to head (66bf3a7). Report is 76 commits behind head on master.

Files Patch % Lines
server/webhook_parser.go 0.00% 0 Missing and 1 partial :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #973      +/-   ##
==========================================
- Coverage   29.65%   29.63%   -0.03%     
==========================================
  Files          52       52              
  Lines        7792     7792              
==========================================
- Hits         2311     2309       -2     
- Misses       5285     5287       +2     
  Partials      196      196              

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar Sep 20 '23 10:09 codecov-commenter

@Kshitij-Katiyar Does this work for Jira Cloud and Server? I'm thinking maybe this was just broken for Jira Cloud?

mickmister avatar Sep 26 '23 17:09 mickmister

@Kshitij-Katiyar Does this need QA review or is this ready for merge?

mickmister avatar Jul 10 '24 00:07 mickmister

@mickmister I think it would be better to get it tested by a QA before merging.

raghavaggarwal2308 avatar Jul 10 '24 12:07 raghavaggarwal2308

@AayushChaudhary0001 The issue should be fixed now. Please verify.

raghavaggarwal2308 avatar Jul 29 '24 06:07 raghavaggarwal2308