Skip to content

Conversation

@pkniazevych
Copy link
Contributor

@pkniazevych pkniazevych commented Jan 21, 2026

✨ PR Description

Purpose: Fix Elementor Pro detection logic in dynamic tag handler to use frontend-specific config instead of app config for accurate plugin status checking.
Main changes:

  • Changed Pro detection from window.elementorAppConfig.hasPro to window.ElementorProFrontendConfig for reliable frontend availability check
  • Updated conditional guard in registerAllyAction to verify frontend configuration object exists before registering URL actions

Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using.
💡 Tip: You can customize your AI Description using Guidelines Learn how

@pkniazevych pkniazevych requested a review from a team January 21, 2026 19:31
@pkniazevych pkniazevych self-assigned this Jan 21, 2026
@bainternet bainternet marked this pull request as draft January 22, 2026 08:35
@bainternet bainternet marked this pull request as ready for review January 22, 2026 08:37
<script>
const registerAllyAction = () => {
if ( ! window?.elementorAppConfig?.hasPro || ! window?.elementorFrontend?.utils?.urlActions ) {
if ( ! window?.ElementorProFrontendConfig || ! window?.elementorFrontend?.utils?.urlActions ) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
if ( ! window?.ElementorProFrontendConfig || ! window?.elementorFrontend?.utils?.urlActions ) {
if ( ! window?.ElementorFrontendConfig || ! window?.elementorFrontend?.utils?.urlActions ) {

Pro is not a requirement, but this is not tested

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's not a requirement, but why would we need this code if there is no Pro?
I found the safest object in the codebase, it's defined by PHP and should be always there.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

because we want it for core users and not just pro

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Discussed in Slack

@pkniazevych pkniazevych merged commit 6bffdd2 into develop Jan 27, 2026
73 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants