Problem/Motivation

The Mautic ECA Drupal module will implement Create, Read, Update, and Delete (CRUD) operations for Mautic contacts using the official ⁠mautic/api-library client. It will expose these operations as ECA actions in Drupal. The module will allow to select from a list of config entity to make the API connection using module mautic_api.

Issue fork mautic_eca-3530306

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

abhisekmazumdar created an issue. See original summary.

abhisekmazumdar’s picture

abhisekmazumdar
he/him
Primary language English
Location India
commented
Title: Implement CRUD for Mautic contacts » Implement CRUD ECA Actions for Mautic contacts

abhisekmazumdar opened merge request !1

abhisekmazumdar’s picture

abhisekmazumdar
he/him
Primary language English
Location India
commented
Related issues: +#3527905: Implement multi-instance Mautic connection config

This work is based on this related issue.

abhisekmazumdar’s picture

abhisekmazumdar
he/him
Primary language English
Location India
commented
Assigned: abhisekmazumdar » Unassigned
Status: Active » Needs review
abhisekmazumdar’s picture

abhisekmazumdar
he/him
Primary language English
Location India
commented

I was not feeling happy with the structure. I made some adjustments to the overall structure to align it with the other actions.
A new commit: https://git.drupalcode.org/project/mautic_eca/-/merge_requests/1/diffs?c...

mallezie’s picture

mallezie
Primary language Dutch
Location Loenhout
commented
Status: Needs review » Needs work
abhisekmazumdar’s picture

abhisekmazumdar
he/him
Primary language English
Location India
commented
Status: Needs work » Needs review

Thank you for the feedback.
I have addressed all the inline comments, please review them.
I also moved the relevant helper API method to mautic_api: #3533406: Add helper method loadConnection & checkApiResponse

mallezie’s picture

mallezie
Primary language Dutch
Location Loenhout
commented

We do not need the json field in every plugin, so removed that and made the needed changes.
Also removed the 2 helper functions, no need to block that on the new issue in mautic_api, that's a good improvement, but can be postponed to later.
There is still one thing missing here, that is the ListContact action, i'll push that tomorrow, if we have that one, then we can start with a setup like delete from mail, by combining that action.

mallezie’s picture

mallezie
Primary language Dutch
Location Loenhout
commented

Added the mautic list action.

This can be used to find a contact via mail, store that output in a token, and then use that input (or part of it) as input token for a get or delete contact.

  • abhisekmazumdar committed 211b5c73 on 1.0.x 
    Issue #3530306 by abhisekmazumdar, mallezie: Implement CRUD ECA Actions...
abhisekmazumdar’s picture

abhisekmazumdar
he/him
Primary language English
Location India
commented
Status: Needs review » Fixed

Thank you for the improvements. It's great to see how everything fits together like Lego pieces.

Status: Fixed » Closed (fixed)

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