Problem/Motivation

Currently in ECA ver 1.0.2, when giving out notifications to user, we cannot change the message type. It would be great to have it in.
The message type: status, warning, or error

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#3 Screenshot 2022-09-12 143735.png35.66 KBjaykumar95

Comments

muaz91 created an issue. See original summary.

jaykumar95’s picture

jaykumar95’s picture

StatusFileSize
new35.66 KB

Drupal: 9.4.5
PHP: 8.1.0
I can confirm that after apply patch from comment-54 message type option added. see in attached image.

NOTE: I have used BPMN modeller for ECA and if there is message action already there then it's not updating the value in select list hence need to create new action.

jurgenhaas’s picture

Status: Active » Fixed

I can confirm that after apply patch from comment-54 message type option added. see in attached image.

Thanks, so this is not an issue with ECA but with the message plugin from Drupal core. So as soon as core implements that feature, it will automatically be available for ECA as well.

NOTE: I have used BPMN modeller for ECA and if there is message action already there then it's not updating the value in select list hence need to create new action.

Yes, if the config footprint of a plugin changes, then the modeller doesn't get that. You can either delete outdated versions of the elements or you could run a drush command which updates all existing models, if any plugins got updated: drush eca:update

@muaz91 thanks for reporting this and thanks @Jaykumar95 for pointing us to the solution.

mxh’s picture

can confirm that after apply patch from comment-54 message type option added. see in attached image.

The linked issue contains a newer patch, any reason why you applied an older one? Your feedback may be very valuable to be posted in that linked issue, to help moving it forward.

jaykumar95’s picture

@mxh I am using Drupal 9.4.5 and that patch #54 passed the test in version 9.4.x , other new patches are for 9.5.x & 10.0.x so I didn't test it on ^9.5 version with new patches.

mxh’s picture

Ah I see now, thanks for clarifying @Jaykumar95

muaz7731’s picture

Thank you all

Status: Fixed » Closed (fixed)

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