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

CRM-21411 Fix issue where unsubscribe group field would not show beca… #11258

Merged
merged 1 commit into from
Nov 21, 2017

Conversation

seamuslee001
Copy link
Contributor

…use CRM.crmMailing.groupNames was empty

Before

unsubscribe group field doesn't show

After

Now it does

Technical Details

because CRM.crmMailing.groupNames is now dynamically populated checking to see if it was set immediately as the JS loaded was causing scope.isUnsubGroupRequired to be set to false incorrectly and hence hiding the unsubscribe group section

Comments

@joannechester @KarinG @eileenmcnaughton could one of you please test this to test it create a CiviMailing from an advanced search and you should see the Unsubscribe Group field show. You may need to disable asset chaching under the debugging settings to ensure the updated code is used.

@seamuslee001
Copy link
Contributor Author

Jenkins re test this please

@joannechester
Copy link
Contributor

Fuzion deployed this to our staging site and, after executing Cleanup Caches the unsubscribe group field is back.

@eileenmcnaughton
Copy link
Contributor

Merging based on @joannechester testing

@eileenmcnaughton eileenmcnaughton merged commit b52f032 into civicrm:4.7.28-rc Nov 21, 2017
@eileenmcnaughton eileenmcnaughton deleted the CRM-21411 branch November 21, 2017 20:33
totten added a commit to totten/uk.co.vedaconsulting.mosaico that referenced this pull request Nov 27, 2017
Upstream made some changes in how the list of groups is supplied (CRM-21411, circa
v4.7.28).  This patch ports the change from civicrm/civicrm-core#11258.

Note the use of `CRM.crmMailing.testGroupNames || CRM.crmMailing.groupNames` is intended to
provide compatibility with older and newer protocols.
sluc23 pushed a commit to ixiam/civicrm-core that referenced this pull request Jan 10, 2018
CRM-21411 Fix issue where unsubscribe group field would not show beca…
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

Successfully merging this pull request may close these issues.

3 participants