[GH-802] Added config setting for getting notification for draft PRs (#804)

* [MM-594] Added config setting for getting notification for draft PRs

* updated help text and display name for GetNotificationForDraftPRs config

Co-authored-by: Doug Lauder <[email protected]>

* improved logic

* get notificaiton for readyForReview in all cases

---------

Co-authored-by: kshitij katiyar <[email protected]>
Co-authored-by: Doug Lauder <[email protected]>
Co-authored-by: kshitij katiyar <[email protected]>

Author: 3 people

plugin.json

@@ -127,6 +127,13 @@
                 "type": "bool",
                 "help_text": "In 'Pushes' event notification, show commit author instead of commit committer.",
                 "default": false
+            },
+            {
+                "key": "GetNotificationForDraftPRs",
+                "display_name": "Get notifications for draft pull requests:",
+                "type": "bool",
+                "help_text": "When set to 'true' you will get a notification with less details when a draft pull request is created and a notification with complete details when they are marked as ready for review. When set to 'false' no notifications are delivered for draft pull requests.",
+                "default": false
             }
         ],
         "footer": "* To report an issue, make a suggestion or a contribution, [check the repository](https://github.com/mattermost/mattermost-plugin-github)."

server/plugin/configuration.go

@@ -42,6 +42,7 @@ type Configuration struct {
 	EnableWebhookEventLogging      bool   `json:"enablewebhookeventlogging"`
 	UsePreregisteredApplication    bool   `json:"usepreregisteredapplication"`
 	ShowAuthorInCommitNotification bool   `json:"showauthorincommitnotification"`
+	GetNotificationForDraftPRs     bool   `json:"getnotificationfordraftprs"`
 }
 
 func (c *Configuration) ToMap() (map[string]interface{}, error) {

server/plugin/webhook.go

@@ -502,8 +502,12 @@ func (p *Plugin) postPullRequestEvent(event *github.PullRequestEvent) {
 		if action == actionOpened {
 			prNotificationType := "newPR"
 			if isPRInDraftState {
+				if !p.configuration.GetNotificationForDraftPRs {
+					return // Draft PR notifications are disabled
+				}
 				prNotificationType = "newDraftPR"
 			}
+
 			newPRMessage, err := renderTemplate(prNotificationType, GetEventWithRenderConfig(event, sub))
 			if err != nil {
 				p.client.Log.Warn("Failed to render template", "error", err.Error())