← All changes | includes/Core/SetupWizard.php +11 -3 3.2.13 → 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)
		@@ -87,8 +88,10 @@
87	88	// phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedHooknameFound -- Reviewed for the NotificationX codebase: acceptable in this context.
88	89	apply_filters( 'nx_frontend_css_version', NOTIFICATIONX_VERSION ),
89	90	'all'
90	91	);
	92	+ // The fonts and icons the themes use are no longer @imported by frontend.css.
	93	+ FrontEnd::get_instance()->enqueue_external_styles();
91	94	}
92	95
93	96	/**
94	97	* Add a body class on the wizard page so the SPA can render a
		@@ -136,12 +139,17 @@
136	139	return $result;
137	140	}
138	141
139	142	/**
140		- * Opt the site into WP Insights usage tracking and send the data to the
141		- * insights API immediately. Triggered when the user proceeds past the
142		- * 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
143	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.
144	152	*
145	153	* @return bool
146	154	*/
147	155	public function optin_tracking() {