search

Miller-Media / send-emails-with-mandrill

A fork of the official wpMandrill plugin for Wordpress, since it is no longer being actively supported.
21 stars 15 forks source link

Potential Conflict with LearnDash Pro Panel plugin #17

Open micaelcovolo opened 5 years ago

micaelcovolo commented 5 years ago

After updating to version 1.2 my dashboard stop working... like the ProPanel from learndash.

MillerMedia commented 5 years ago

@micaelcovolo Can you provide some more details? What exactly stops working? Does it not load at all or are you getting errors on the screen?

micaelcovolo commented 5 years ago

I'm using also the LearnDash Pro Panel plugin... so after your plugin update all dashboard panels (Learndash) stopped working.

Em seg, 18 de fev de 2019 às 14:41, Matt Miller notifications@github.com escreveu:

@micaelcovolo https://github.com/micaelcovolo Can you provide some more details? What exactly stops working? Does it not load at all or are you getting errors on the screen?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Miller-Media/send-emails-with-mandrill/issues/17#issuecomment-464824028, or mute the thread https://github.com/notifications/unsubscribe-auth/Atiro_ODUWYRfl4rwMz5ejR51pgKUdBmks5vOuW9gaJpZM4bBUPP .

MillerMedia commented 5 years ago

Were there any other plugins that you have updated? Are you seeing errors directly related to the our plugin? If so, could you please share?

Our latest plugin updates were confined simply to our plugin's admin panel and the e-mail sending functionality of Mandrill.

micaelcovolo commented 5 years ago

Let try to explain better ... all was working fine ... then I’ve updated only your plugin... so far so good... but when I went to my dashboard I notice that all the dashboard windows was loading forever... including yours ... deactivating your plugin... all fine again. I’m not sure if is your plugin but I’m sure that without it is working fine

Il giorno lun 18 feb 2019 alle 15:45 Matt Miller notifications@github.com ha scritto:

Were there any other plugins that you have updated? Are you seeing errors directly related to the our plugin? If so, could you please share?

Our latest plugin updates were confined simply to our plugin's admin panel and the e-mail sending functionality of Mandrill.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Miller-Media/send-emails-with-mandrill/issues/17#issuecomment-464840582, or mute the thread https://github.com/notifications/unsubscribe-auth/Atiro58Ouu350Sx5biAxUQdzzUICrsE8ks5vOvTKgaJpZM4bBUPP .

MillerMedia commented 5 years ago

Ok this makes sense. Seems to imply it has something to do with ours if everything does get resolved when you deactivate our plugin.

If you're comfortable with it, it would be great to take a deeper look into the issue which would require having temporary access to your site's admin dashboard. If you're not comfortable with that, I could offer up some steps that you could walk through to help us get some better debugging information. Let me know if you'd be open to either of those so we can determine that problem and fix it ASAP. Thanks!

andrewhilson commented 5 years ago

I think the issue with this is that the Mandrill reports API no longer accepts URL tracking. I submitted a PR with this which fixed it for me, but I think the entire reporting-on-dashboard feature should be removed.

It can take a very long time for a report API call to execute which leads to page hanging - and on a page frequently used such as the dashboard.....