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

feat: Analytics architecture for multiple libraries #91

Merged
merged 1 commit into from
Dec 4, 2023

Conversation

omerhabib26
Copy link
Contributor

@omerhabib26 omerhabib26 commented Nov 13, 2023

Description

  • Introduced a way to implement multiple analytics libraries with only a little effort
  • Write a base class to manage analytics with limited methods
  • Also added the support to add analytics using feature flag implementation

@omerhabib26 omerhabib26 changed the title [Draft/Spike]feat: Analytics architecture for multiple libraries feat: Analytics architecture for multiple libraries Nov 24, 2023
@omerhabib26 omerhabib26 marked this pull request as ready for review November 24, 2023 17:26
@omerhabib26 omerhabib26 force-pushed the omer/LEARNER-9667 branch 2 times, most recently from 445f4e5 to 3a8df9c Compare November 30, 2023 13:01
- Introduced a way to implement multiple analytics libraries with only a little effort
- Write a base class to manage analytics with limited methods
- Also added the support to add analytics using feature flag implementation

fix: LEARNER-9667
@omerhabib26
Copy link
Contributor Author

@volodymyr-chekyrta this PR is approved from our side.

@volodymyr-chekyrta volodymyr-chekyrta merged commit e0577f9 into openedx:develop Dec 4, 2023
1 check passed
@omerhabib26 omerhabib26 deleted the omer/LEARNER-9667 branch December 4, 2023 11:24
k1rill pushed a commit to touchapp/openedx-app-android that referenced this pull request Dec 4, 2023
- Introduced a way to implement multiple analytics libraries with only a little effort
- Write a base class to manage analytics with limited methods
- Also added the support to add analytics using feature flag implementation

fix: LEARNER-9667
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.

4 participants