Assign A Different Slack Channel To Each Content Type

Well, as of now this will gonna do the trick but it would be much better if we have more structured data so that it will be less confusing

It is working as expected, but while Testing observed two issues:

1. As a user, once I add the channel name for any content type, then after I'm not able to remove(empty) the earlier added channal name
2. Change the label of slack channel name input box, it should be unique. For example: For Article Content-Type, Select the channel for Article Content-Type. Refer screenshot

Hi @Lal_, Please fix above mentioned issue.

@ambuj30oct good catch

quick fix

Tested and verified the module after adding the patch created by @Lal_. And it is working as expected.

@Lal_ As per functionality-wise patch looks good but there are coding standard issues. Please fix them.

FILE: ...en_source/modules/contrib/notify_to_slack/notify_to_slack.module
----------------------------------------------------------------------
FOUND 4 ERRORS AFFECTING 1 LINE
----------------------------------------------------------------------
 61 | ERROR | [x] Expected 1 space before "?"; 0 found
 61 | ERROR | [x] Expected 1 space after "?"; 0 found
 61 | ERROR | [x] Expected 1 space before ":"; 0 found
 61 | ERROR | [x] Expected 1 space after ":"; 0 found
----------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------


FILE: ...source/modules/contrib/notify_to_slack/src/Form/SettingsForm.php
----------------------------------------------------------------------
FOUND 6 ERRORS AND 1 WARNING AFFECTING 6 LINES
----------------------------------------------------------------------
  97 | ERROR   | [x] Expected 1 space after closing parenthesis;
     |         |     found 0
 100 | WARNING | [ ] Translatable strings must not begin or end with
     |         |     white spaces, use placeholders with t() for
     |         |     variables
 101 | ERROR   | [x] There should be no white space after an opening
     |         |     "("
 101 | ERROR   | [x] Concat operator must be surrounded by a single
     |         |     space
 110 | ERROR   | [ ] If the line declaring an array spans longer than
     |         |     80 characters, each element should be broken
     |         |     into its own line
 194 | ERROR   | [x] Expected 1 space after closing parenthesis;
     |         |     found 0
 195 | ERROR   | [x] Line indented incorrectly; expected 6 spaces,
     |         |     found 8
----------------------------------------------------------------------
PHPCBF CAN FIX THE 5 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------

oops missed that