Steps to Reproduce:
1. Go to the module configuration.
2. Select any entity(content).
3. Then select the notification operation for that entity. And Save the configuration.
4. Create a selected entity(node). And check

Actual Result:
1. When we create/update any entity type, then it gives us an error.
2. When Custom slack Message is not configured, then default message notification not working.

Expected Result:
1. When we create/update any entity type, then it should not throws any error.
2. When Custom Slack Message for the entity is not configured, then the user should receive the default message.

CommentFileSizeAuthor
#7 3102394-7.patch423 bytessnehalgaikwad
error-2.png311.35 KBambuj_gupta
error_1.png317.14 KBambuj_gupta

Comments

ambuj30oct created an issue. See original summary.

Rangaswini’s picture

Assigned: Unassigned » Rangaswini
Lal_’s picture

Priority: Normal » Major

pushing this to major...

Lal_’s picture

Assigned: Rangaswini » Unassigned

@rangaswini please assign it back if you are still working on a patch

Lal_’s picture

Issue tags: +DIACWApril2020
snehalgaikwad’s picture

Assigned: Unassigned » snehalgaikwad
snehalgaikwad’s picture

Assigned: snehalgaikwad » Unassigned
Status: Active » Needs review
StatusFileSize
new423 bytes

Made the required change in parameters.

Status: Needs review » Needs work

The last submitted patch, 7: 3102394-7.patch, failed testing. View results

Lal_’s picture

Status: Needs work » Reviewed & tested by the community

I think this is happening due to test, which is a task for another day

  • Lal_ committed ea4b24d on 8.x-2.x authored by snehalgaikwad
    Issue #3102394 by snehalgaikwad, ambuj30oct, Lal_: When we create/update...
Lal_’s picture

Status: Reviewed & tested by the community » Fixed

good job guys

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.