search

inboundnow / retired-landing-pages

Create landing pages for your WordPress site. Monitor and improve conversion rates, run A/B split tests, customize your own templates and more.
http://www.inboundnow.com/landing-pages/
GNU General Public License v2.0
4 stars 2 forks source link

ACF Pro issue with repeater fields #287

Closed atwellpub closed 8 years ago

atwellpub commented 8 years ago

Ran into a strange issue where our current setup did not work well with a template created by a 3rd party. I was able to fix it (possibly temporarily) for the customer issue. I am not understanding 100% why it was required.

atwellpub commented 8 years ago

https://github.com/inboundnow/landing-pages/blob/develop/classes/class.acf-integration.php#L308-L311

daprela commented 8 years ago

Have you verified if the template uses the repeater in a 'standard' way? Maybe it is a way of using it that you didn't anticipate and that creates this issue, like, for example, many levels of nested repeater fields. It's just a guess, I thought that in my templates I had used it in pretty much all of the possible ways but I could be wrong

atwellpub commented 8 years ago

I'm just not sure, Giulio. I had to make updates again today to get Ginger to quite a bunch of soft errors. Commit above.