Internal: Suppress Hello Pro banner when Elementor Core is active [TMZ-1072] - #687
Open
Ntnelbaba wants to merge 2 commits into
Open
Internal: Suppress Hello Pro banner when Elementor Core is active [TMZ-1072]#687Ntnelbaba wants to merge 2 commits into
Ntnelbaba wants to merge 2 commits into
Conversation
…-24302] Hide the Hello theme conversion banner whenever Core is installed so users only see Core's upgrade notice, and drop the duplicate Pro welcome config. Ref: ED-24302 Co-authored-by: Cursor <cursoragent@cursor.com>
Hello Elementor BuildLast updated at: 20260824.1205 ✅ Hello Elementor build is ready for download. You can download the latest build from the link below: The build is available for 7 days. |
Ref: TMZ-1072 Co-authored-by: Cursor <cursoragent@cursor.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Jira
TMZ-1072 — Pro banner is duplicated when Core and Hello exists
Test plan
#e-conversion-bannerappears on WP admin pages (e.g. Pages list, Plugins), not Hello's#ehe-admin-cbconversion-banner-layout.test.ts✨ PR Description
1. Problem & Context
Suppress the Hello Pro upsell banner when Elementor Core is active—currently the banner only hides when Pro is installed AND Core is active, but should hide for either condition (ED-24302).
2. What Changed (Where)
conversion-banner.php: Changed condition from AND to OR—now suppresses banner if Pro exists OR Core is activeadmin-config.php: Removed hardcoded Pro upsell welcome box and$has_provariable entirelyconversion-banner-layout.test.ts: Updated test to reflect new behavior; simplified mock setup3. How It Works
Entry point is
is_conversion_banner_active(). Previously required bothhas_pro() && is_elementor_active()to suppress; now requires only one condition met. Upsell content inadmin-config.phpis now unconditionally removed for active Elementor installs, delegating suppression logic to the banner component.4. Risks
None—logic is more permissive (shows fewer banners), and
is_elementor_active()is an existing utility. Test changes are straightforward and cover the actual behavior now.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