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

Export Emails to CVS: Exclude Inactive Groups #1865

Merged
merged 26 commits into from
Mar 4, 2017
Merged

Conversation

DawoudIO
Copy link
Contributor

@DawoudIO DawoudIO commented Feb 8, 2017

@DawoudIO DawoudIO added this to the 2.6.0 milestone Feb 8, 2017
@DawoudIO DawoudIO changed the title moved group to ORM Export Email to CVS Cleanup Feb 8, 2017
@DawoudIO DawoudIO modified the milestones: 2.7.0, 2.6.0 Feb 11, 2017
@crossan007
Copy link
Contributor

crossan007 commented Feb 25, 2017

Why would we only want some groups to be enabled for email export (grp_enable_email_export
) ? is it for mailchimp?

Are you preparing to enabled automatic export / synchronization with MailChimp?

@DawoudIO
Copy link
Contributor Author

I we want to email sunday school parents etc, but at the same time we have old xmas plays that are no longer valid, but want them around

@DawoudIO DawoudIO changed the title Export Email to CVS Cleanup Export Emails to CVS & Exclude Inactive Groups Feb 27, 2017
@DawoudIO DawoudIO changed the title Export Emails to CVS & Exclude Inactive Groups Export Emails to CVS: Exclude Inactive Groups Feb 27, 2017
@DawoudIO
Copy link
Contributor Author

This is ready for testing

  1. Import Demo data
  2. export an email to cvs / review data
  3. Disable a group merge to master #1
  4. export an email to cvs / review data notice merge to master #1 group is missing
  5. exclude a group Live demo not working #2 from email export
  6. export an email to cvs / review data notice Live demo not working #2 group is missing

@crossan007 crossan007 merged commit bfbcc67 into develop Mar 4, 2017
@crossan007 crossan007 deleted the email-export-cleanup branch March 4, 2017 13:04
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.

2 participants