| @@ -83,8 +83,9 @@ | ||
| 83 | 83 | wp_enqueue_style( |
| 84 | 84 | 'notificationx-public', |
| 85 | 85 | Helper::file( 'public/css/frontend.css', true ), |
| 86 | 86 | [], |
| 87 | + // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedHooknameFound -- Reviewed for the NotificationX codebase: acceptable in this context. | |
| 87 | 88 | apply_filters( 'nx_frontend_css_version', NOTIFICATIONX_VERSION ), |
| 88 | 89 | 'all' |
| 89 | 90 | ); |
| 90 | 91 | } |
| @@ -135,12 +136,17 @@ | ||
| 135 | 136 | return $result; |
| 136 | 137 | } |
| 137 | 138 | |
| 138 | 139 | /** |
| 139 | - * Opt the site into WP Insights usage tracking and send the data to the | |
| 140 | - * insights API immediately. Triggered when the user proceeds past the | |
| 141 | - * Welcome step — the in-card notice states that proceeding consents to | |
| 140 | + * Record the user's opt-in to WP Insights usage tracking and send the data | |
| 141 | + * to the insights API immediately. Triggered when the user proceeds past | |
| 142 | + * the Welcome step — the in-card notice states that proceeding consents to | |
| 142 | 143 | * collecting the admin email to personalise the setup. |
| 144 | + * | |
| 145 | + * The wizard no longer decides *whether* data is collected: collection is | |
| 146 | + * enabled from the backend on activation regardless of this flow (see | |
| 147 | + * {@see \NotificationX\Admin\PluginInsights::is_tracking_allowed()}). | |
| 148 | + * This only stores the explicit consent state and sends immediately. | |
| 143 | 149 | * |
| 144 | 150 | * @return bool |
| 145 | 151 | */ |
| 146 | 152 | public function optin_tracking() { |
