Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: analytics screen events improvements and name fixes #76

Merged
merged 2 commits into from
Sep 13, 2024

Conversation

saeedbashir
Copy link

@saeedbashir saeedbashir commented Sep 9, 2024

This PR addresses the following issues:

  1. Correct the naming of payments events as per the other events like no space after : like the event Payments: Upgrade Now Clicked changed to Payments:Upgrade Now Clicked.
  2. Correct/fix the calling of screen events and introduce new events where needed.

Jira Tickets:

  1. LEARNER-10208
  2. LEARNER-10209

Screen events sheet: https://docs.google.com/spreadsheets/d/1qf0sboa6CXGhsh76eERCJM8FpUWCUWr3MSItLuRtdNE/edit?gid=635851976#gid=635851976

@saeedbashir saeedbashir requested a review from rnr September 12, 2024 01:49
Copy link

@shafqat-muneer shafqat-muneer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍
I conducted tests on a few sample events during the live session, and they are being recorded as expected.

Screen.Recording.2024-09-12.at.3.48.07.PM.mov

@saeedbashir saeedbashir merged commit bd5bd08 into 2U/develop Sep 13, 2024
3 checks passed
@saeedbashir saeedbashir deleted the saeed/analytics_fixes branch September 13, 2024 05:01
rnr added a commit that referenced this pull request Dec 9, 2024
* fix: video tab with youtube (#52)

* fix: don't show error if only youtube videos

* fix: call UI updates from main queue

* fix: don't send completion at youtube video start

---------

Co-authored-by: Anton Yarmolenko <[email protected]>

* chore: delete unneeded MainActor instructions

* feat: save playback speed (#67)

* chore: save playback speed in CourseStorage

* chore: add full screen for youtube on iPad

* chore: save value in userdefaults

* chore: little refactor

* fix: fix warning

* chore: added CourseStorage as injection

* chore: moved playback speed to user settings

---------

Co-authored-by: Anton Yarmolenko <[email protected]>

* chore: analytics screen events improvements and name fixes (#76)

* chore: fix tests

---------

Co-authored-by: Anton Yarmolenko <[email protected]>
Co-authored-by: Saeed Bashir <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants