search

PopupMaker / Popup-Maker

Popup Maker plugin for WordPress
https://wppopupmaker.com/
104 stars 38 forks source link

CF7 Update breaks integration #851

Closed danieliser closed 4 years ago

danieliser commented 4 years ago

Describe the bug

The latest CF7 update removed their jQuery event system and broke the integration we have with their success events.

Requires replacing our events with the new DOM events.

Additional context

DevSnapTech-Lindsay commented 4 years ago

Is there an ETA when you will have a patch for this or a detailed solution on how we can edit the code in the meantime to fix this?

fpcorso commented 4 years ago

@DevSnapTech-Lindsay At quick glance, all of our CF7 JS code is within our contactform7.js file. And, there is only one jQuery event in that file: https://github.com/PopupMaker/Popup-Maker/blob/master/assets/js/src/integration/contactform7.js#L9

So, I think it may only need to be updated to the new CF7 DOM events and maybe update the code for getting the form ID and instance ID.

I'll be looking into this issue today and will let you know if there is anything more than that.