1049 bug no events triggered before gameplay despite video start

[prakash-bhatt-yral]

No description provided.

[Copilot]

Pull Request Overview

This PR addresses bug #1049 by refactoring storage implementation and adding an initialization event to track when the application starts. The changes replace local storage with cookie-based storage for device ID management and ensure events are properly triggered at application startup.

• Replaced LocalStorage implementation with Cookie-based Storage for persistent device ID management
• Added an init_event function to track application initialization with user principal data
• Enhanced logging for debugging Mixpanel event flow

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
ssr/src/utils/src/storage.rs	New cookie-based storage implementation replacing local storage
ssr/src/utils/src/mixpanel/mixpanel_events.rs	Added init_event function and updated device ID references
ssr/src/utils/src/local_storage.rs	Removed local storage implementation
ssr/src/utils/src/lib.rs	Updated module exports to use storage instead of local_storage
ssr/src/page/src/root.rs	Added init_event call on page load with user principal
ssr/src/page/src/post_view/mod.rs	Changed Effect to Effect::new_isomorphic for better SSR compatibility

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

The function logic is inconsistent with the removed LocalStorage implementation. The original code checked uuid_value.is_empty() for an empty string, but this code checks for None. This could cause issues if cookies return empty strings instead of None, leading to unnecessary UUID regeneration.

Suggested change

	if let Some(uuid) = uuid_value {
	uuid
	if let Some(ref uuid) = uuid_value {
	if !uuid.is_empty() {
	uuid.clone()
	} else {
	let new_device_id = uuid::Uuid::new_v4().to_string();
	set_uuid.set(Some(new_device_id.clone()));
	new_device_id
	}

[Copilot]

Missing closing quote in the log message format string. It should be "Init event {:?})" → "Init event {:?}".

Suggested change

	leptos::logging::log!("Init event {:?})", user_principal);
	leptos::logging::log!("Init event {:?}", user_principal);

[Copilot]

This debug log message is misleading as it appears in the general track_event_server_fn function, not specifically for init events. This will log for all events tracked through this function, which could be confusing when debugging.

Suggested change

	leptos::logging::log!("Init event served");
	leptos::logging::log!("Event tracked via track_event_server_fn");