search

openedx / openedx-app-ios

The mobile app for iOS for the Open EdX Platform.
Apache License 2.0
19 stars 13 forks source link

feat: implementation / enhancements of analytics #348

Closed saeedbashir closed 3 months ago

saeedbashir commented 3 months ago

This PR adds analytics to most parts of the app, as requested by the product team (@moiz994).

Note: @marcotuts We have discussed it internally, and I have renamed the already implemented analytics names for consistency between already implemented and newly implemented analytics, e.g Forgot_password_Clicked to Logistration_Forgot_Password_Clicked, Cookie_Policy_Clicked to Profile_Cookie_Policy_Clicked.

saeedbashir commented 3 months ago

@volodymyr-chekyrta The tests are passing locally but failing on the PR, and I'm not getting any useful insights from the logs. Could you please look into it and see what needs to be fixed.

saeedbashir commented 3 months ago

LGTM perhaps needs to coordinate with @shafqat-muneer and add analytics for Calendar sync too (maybe after his PR merging?)

We have created a separate task for calendar integration events.