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

Save information that data were exported #157

Closed
scardinius opened this issue Aug 2, 2018 · 6 comments
Closed

Save information that data were exported #157

scardinius opened this issue Aug 2, 2018 · 6 comments
Assignees
Milestone

Comments

@scardinius
Copy link
Contributor

scardinius commented Aug 2, 2018

Because of GDPR rules we have to save information that some data were exported. Those could be contact, activity or contribution objects.

I'm going to prepare solution for action "Export contacts", "Export actiivities" etc.
Each exported object will have a new type activity, for example "Contact exported", "Activity exported" with additional fields as id of object, subject, who exported.

It's necessary to use new type of hook, because hook_civicrm_export doesn't have ids of objects
scardinius/civicrm-core@f46c373

So at this branch caltha-eu:dev-157 is my proposal
https://github.com/veda-consulting/uk.co.vedaconsulting.gdpr/compare/master...caltha-eu:dev-157?expand=1

scardinius added a commit to caltha-eu/uk.co.vedaconsulting.gdpr that referenced this issue Aug 2, 2018
scardinius added a commit to caltha-eu/uk.co.vedaconsulting.gdpr that referenced this issue Aug 2, 2018
scardinius added a commit to caltha-eu/uk.co.vedaconsulting.gdpr that referenced this issue Aug 2, 2018
scardinius added a commit to caltha-eu/uk.co.vedaconsulting.gdpr that referenced this issue Aug 2, 2018
scardinius added a commit to caltha-eu/uk.co.vedaconsulting.gdpr that referenced this issue Aug 2, 2018
scardinius added a commit to caltha-eu/uk.co.vedaconsulting.gdpr that referenced this issue Aug 2, 2018
scardinius added a commit to caltha-eu/uk.co.vedaconsulting.gdpr that referenced this issue Aug 2, 2018
scardinius added a commit to caltha-eu/uk.co.vedaconsulting.gdpr that referenced this issue Aug 2, 2018
scardinius added a commit to caltha-eu/uk.co.vedaconsulting.gdpr that referenced this issue Aug 2, 2018
scardinius added a commit to caltha-eu/uk.co.vedaconsulting.gdpr that referenced this issue Aug 2, 2018
scardinius added a commit to caltha-eu/uk.co.vedaconsulting.gdpr that referenced this issue Aug 2, 2018
scardinius added a commit to caltha-eu/uk.co.vedaconsulting.gdpr that referenced this issue Aug 2, 2018
scardinius added a commit to caltha-eu/uk.co.vedaconsulting.gdpr that referenced this issue Aug 2, 2018
scardinius added a commit to caltha-eu/uk.co.vedaconsulting.gdpr that referenced this issue Aug 2, 2018
@veda-consulting
Copy link
Member

Hi @scardinius
Do you feel these changes are production ready? We are due to release a new version of the GDPR extension in the next week and would like to include this functionality if possible?

@scardinius
Copy link
Contributor Author

Hi @veda-consulting
this feature needs more testing from our side although it looks quite good.

Apart from that it relies on new civicrm hook scardinius/civicrm-core@f46c373 which is not included into civicrm core yet. I think we should wait until CRM_Utils_Hook::exportIds will be official hook.

@scardinius
Copy link
Contributor Author

Hi @veda-consulting I've just published my proposal for new type of hook needed for this feature
https://lab.civicrm.org/dev/core/issues/457

@scardinius
Copy link
Contributor Author

Hi @veda-consulting I have a good news :-) Core team approved my modifications to export hook
https://lab.civicrm.org/dev/core/issues/457 and they will be available from CiviCRM 5.8

PR #176 is ready to use

@scardinius
Copy link
Contributor Author

Hi @veda-consulting I created a fix for this feature because civicrm core changed a bit :-)

@tapashdatta
Copy link
Collaborator

Fixed in recent release.

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

No branches or pull requests

4 participants