| @@ -745,22 +745,19 @@ | ||
| 745 | 745 | public function should_display_nps_survey_notice() { |
| 746 | 746 | |
| 747 | 747 | $is_store_checkout_imported = (bool) get_option( '_cartflows_wizard_store_checkout_set', false ); |
| 748 | 748 | $onboarding_completed = (bool) get_option( 'wcf_setup_complete', false ); |
| 749 | - $is_first_funnel_imported = (bool) get_option( 'wcf_first_flow_imported', false ); | |
| 750 | 749 | $total_funnels = $this->get_published_flow_count(); |
| 751 | 750 | |
| 752 | 751 | /** |
| 753 | - * Show the notice in any of these conditions: | |
| 752 | + * Show the notice in either of these conditions: | |
| 754 | 753 | * 1. User finished the onboarding wizard AND imported the store checkout funnel. |
| 755 | - * 2. User imported their first funnel AND has at least one published funnel. | |
| 756 | - * 3. User has at least one published funnel (built manually). | |
| 757 | - * | |
| 758 | - * Note: condition 3 intentionally has NO upper bound on funnel count, | |
| 759 | - * so power users with many funnels still see the survey. | |
| 754 | + * 2. User has at least one published funnel — imported or built by hand. This has | |
| 755 | + * NO upper bound on funnel count, so power users still see the survey, and it | |
| 756 | + * subsumes the old "imported their first funnel" branch. | |
| 760 | 757 | */ |
| 761 | 758 | return ( true === $is_store_checkout_imported && true === $onboarding_completed ) |
| 762 | - || ( true === $is_first_funnel_imported && ! empty( $total_funnels ) && 1 >= $total_funnels ); | |
| 759 | + || ! empty( $total_funnels ); | |
| 763 | 760 | } |
| 764 | 761 | } |
| 765 | 762 | |
| 766 | 763 | Cartflows_Admin_Notices::get_instance(); |