[MM-507,699]: Added feature to receive notifications for different Jira actions.

[Kshitij-Katiyar]

Summary

• To get a DM notification from the Jira plugin, the user has to meet certain criteria like being assigned or mentioned. But now even if a user is a reporter or watching that issue, he will still get a DM notification.
• This PR contains all the changes from Pr #872 & Pr #840 So this PR alone is enough to fix this issue so we can close Pr #872 & Pr #840 afterward.
• Updated commands for notifications are:

1. /jira instance settings notification reporter on
2. /jira instance settings notification watching on
3. /jira instance settings notification assignee on
4. /jira instance settings notification mention on

Screenshots

Watching

Reporter

Invalid command

Issue

• Fixes Reporter should receive DM when someone comments on their issue #699
• Fixes Add support for DM subscriptions based on "watching" an issue in Jira #507
• Pr #872

[esethna]

Thanks @Kshitij-Katiyar! A few comments:

1. Can we remove the term "because" in the notifications:
   
2. Can you please share a screenshot of the /jira help command? If not done already we should add these new options to that response
3. The screenshot shows "task", but is this term dynamic depending on the ticket type? (ie story, bug etc?)
4. What is the default for these, and if someone runs /jira notifications on/off without the role specified, what happens to the values of the notifications watching and notifications reporter? I'm trying to figure out how we treat these settings, ie as subsettings of "notifications" as a whole, or treated as separate entities that are always disconnected from the state of notifications for being assigned and mentioned.

cc// @asaadmahmood if you have thoughts here?

[raghavaggarwal2308]

@esethna

1. Removed
   
   

2. Here is the screenshot of help command:
   

3. Yes, the value is dynamic

4. If we skip the role in the command or enter invalid value, we will get a message
   
   
   

[fmartingr]

I've done a quick review mostly because I'm not super experienced with this plugin. Logic seems good, just pointed out a few nitpicks.