• Resolved markofapproval

    (@markofapproval)


    Hey guys,

    Excited to see you rebuilding this plugin in such a modular way. Thank you!

    One observation I’ve made is that I cannot toggle the conditional popup for Locations on layouts and Scripts or Styles.

    This issue seems to stem from a JS error that is caught by React and seems to be conflicting with the plugin PODS.

    Are you able to verify this at your end?

    Picture of Error Message

    When PODS is disabled, it works.

    Very best,
    Mark.

Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Author Tangible

    (@tangibleinc)

    Thank you for the issue description – the screenshot was very helpful.

    It was indeed a compatibility issue with PODS. This will be fixed in the next version release, hopefully soon.

    If you have any further questions or feedback, please feel free to join our discussion forum.

    What specific compatibility issue was this? From the Pods side, is there anything we can do to improve this?

    Plugin Author Eliot Akira

    (@miyarakira)

    Hi Scott, thank you for the offer to help.

    The issue was mostly on our plugin’s side. When Pods is active, it enqueues wp.element in the admin, including in our plugin’s settings pages, which use Preact instead. This caused an unexpected situation, somewhat like having two different instances of React. The solution was to improve a certain condition used by our React components, to prioritize Preact over wp.element if both are loaded on the page.

    I suppose it might help if Pods enqueued wp.element only on the admin screens where it’s used. But maybe that’s unnecessary, since wp.element is loaded on most admin screens anyway.

    Thread Starter markofapproval

    (@markofapproval)

    Thank you it seems to be working well now! Thank you for looking into this.

    But I am troubleshooting another issue and will report on another thread!

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘React JS Error when PODS Installed’ is closed to new replies.