-
Notifications
You must be signed in to change notification settings - Fork 44
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
4.5 compatible? #88
Comments
@emilyf don't see any hurdles, but we haven't really tested with v4.5. |
I've just installed in 4.5 and the menu items don't appear to work. They are in the menu structure as expected, but "Mailchimp settings" links to /civicrm/mailchimp/settings_reset_1 and just drops me back at the Home screen. |
I have installed in 4.5 as well and also got the same "Mailchimp settings" linking to /civicrm/mailchimp/settings_reset_1. However, if I change the link to /civicrm/mailchimp/settings?reset=1 I get the correct page. |
I have this issue too. I don't really know how the menu URLs are formed, but I looked in the database and they appear to be correct in the navigation table. I tried to rebuild menus, no improvement. Cleared caches, no improvement. Where to fix this? |
@robbrandt, the lastest code in github has this corrected. There are a couple of other fixes in as well. You may want to try using a current pull. (I know the code up through the end of last week is working on one of my platforms. Some new code went up this week... haven't had a chance to check it out yet.) |
That does solve the menu issues, thanks. |
Closign this one as latest release (1.4) already solves this. |
Just wondering if this is/will be 4.5 compatible? The extension page says 4.4 only. Thanks!
The text was updated successfully, but these errors were encountered: