Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update civicrm_rules_event.inc #536

Open
wants to merge 1 commit into
base: 7.x-master
Choose a base branch
from
Open

Conversation

yurg
Copy link
Contributor

@yurg yurg commented Jul 22, 2018

There are 3 Rule Events available for CiviCRM Contact:

  • After updating an existing civicrm contact
  • CiviCRM Contact has been updated
  • Contact has been updated

A little bit hard to find what is the difference. Let us help users by telling that "Contact has been updated" comes from Event.
Perhaps, would be even better to move this one under CiviCRM Events option group.

There are 3 Rule Events available for CiviCRM Contact: 

* After updating an existing civicrm contact
* CiviCRM Contact has been updated
* Contact has been updated

A little bit hard to find what is the difference. Let us help users by telling that "Contact has been updated" comes from Event.
Perhaps, would be even better to move this one under CiviCRM Events option group.
@civicrm-builder
Copy link

Can one of the admins verify this patch?

@civibot
Copy link

civibot bot commented Jul 22, 2018

(Standard links)

@jackrabbithanna
Copy link
Contributor

'CiviCRM Contact has been updated' is coming from the CiviCRM Entity module...

@yurg
Copy link
Contributor Author

yurg commented Jul 23, 2018

@jackrabbithanna Thank you for note! The only mystery to resolve is, where 'After updating an existing civicrm contact' comes from (guess is form CiviCRM-Drupal too though ) and how is it differs from 'CiviCRM Contact has been updated' and why do we need both.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants