Skip to content

Add support to desktop navigation device#83

Merged
defagos merged 7 commits intodevelopfrom
feature/desktop-device
Jan 25, 2024
Merged

Add support to desktop navigation device#83
defagos merged 7 commits intodevelopfrom
feature/desktop-device

Conversation

@pyby
Copy link
Member

@pyby pyby commented Dec 17, 2023

Description

This PR adds desktop support for navigation_device, according to SRGAnalytics-documentation (on all events: pageView, media events and events).

Inspired by SRGSSR/pillarbox-apple#364.

Changes made

  • Use isMacCatalystApp and isiOSAppOnMac on NSProcessInfo to set desktop value for navigation_device property.
  • Add missing common properties in tests.

Apply suggestions from code review

Co-authored-by: Walid Kayhal <[email protected]>
@pyby pyby requested a review from defagos January 8, 2024 12:13
pyby and others added 3 commits January 8, 2024 13:58
Co-authored-by: Samuel Défago <[email protected]>
(cherry picked from commit 8c8cbe7227a1c9af364ba67544dbaa7837d1fd69)
@defagos defagos force-pushed the feature/desktop-device branch 3 times, most recently from f838640 to b0ebf9b Compare January 24, 2024 17:25
@defagos defagos merged commit 8251819 into develop Jan 25, 2024
@defagos defagos deleted the feature/desktop-device branch January 25, 2024 09:36
@defagos
Copy link
Member

defagos commented Jan 25, 2024

Forced squash & merge. The CI currently has issues but tests run fine locally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants