← All changes | includes/Core/SetupWizard.php +12 -3 3.2.9 → 3.3.3 View file →

		@@ -11,8 +11,9 @@
11	11
12	12	namespace NotificationX\Core;
13	13
14	14	use NotificationX\Admin\Admin;
	15	+use NotificationX\FrontEnd\FrontEnd;
15	16	use NotificationX\GetInstance;
16	17
17	18	/**
18	19	* @method static SetupWizard get_instance($args = null)
		@@ -83,11 +84,14 @@
83	84	wp_enqueue_style(
84	85	'notificationx-public',
85	86	Helper::file( 'public/css/frontend.css', true ),
86	87	[],
	88	+ // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedHooknameFound -- Reviewed for the NotificationX codebase: acceptable in this context.
87	89	apply_filters( 'nx_frontend_css_version', NOTIFICATIONX_VERSION ),
88	90	'all'
89	91	);
	92	+ // The fonts and icons the themes use are no longer @imported by frontend.css.
	93	+ FrontEnd::get_instance()->enqueue_external_styles();
90	94	}
91	95
92	96	/**
93	97	* Add a body class on the wizard page so the SPA can render a
		@@ -135,12 +139,17 @@
135	139	return $result;
136	140	}
137	141
138	142	/**
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
	143	+ * Record the user's opt-in to WP Insights usage tracking and send the data
	144	+ * to the insights API immediately. Triggered when the user proceeds past
	145	+ * the Welcome step — the in-card notice states that proceeding consents to
142	146	* collecting the admin email to personalise the setup.
	147	+ *
	148	+ * The wizard no longer decides *whether* data is collected: collection is
	149	+ * enabled from the backend on activation regardless of this flow (see
	150	+ * {@see \NotificationX\Admin\PluginInsights::is_tracking_allowed()}).
	151	+ * This only stores the explicit consent state and sends immediately.
143	152	*
144	153	* @return bool
145	154	*/
146	155	public function optin_tracking() {