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: FullStory Integration and Analytics Implementation #471

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

saeedbashir
Copy link
Contributor

This PR adds support for the FullStory SDK. The FullStory analytics and pages are integrated. FullStory added as an analytics service so all the events will be sent to the FullStory when configured.

This PR adds new event type i.e. screen events. As some the analytics providers explicitly provided screen capturing events. For example, segment.io, FullStory. For Firebase, a normal track event will be fired on calling a trackScreenEvent.

Integration Guidelines: https://help.fullstory.com/hc/en-us/articles/8867138701719-Integrating-Fullstory-into-a-SwiftUI-App

Data Capture GuideLines:
https://help.fullstory.com/hc/en-us/articles/8867138701719-Integrating-Fullstory-into-a-SwiftUI-App

Config:

FULLSTORY:
    ENABLED: true
    ORG_ID: 'org_id'

Xcode Settings: A new user-defined setting FULLSTORY_ENABLED added in the Xcode settings. This is added because FullStory the initialization of depends on build time script /FullStory/tools/FullStoryCommandLine and if the keys aren't available in the info.plist (disabled fullstory), it fails the project build. The white-label script will update this new setting FULLSTORY_ENABLED depending on the config values.

@saeedbashir saeedbashir requested a review from rnr June 24, 2024 10:08
rnr
rnr previously approved these changes Jun 24, 2024
@volodymyr-chekyrta
Copy link
Contributor

To flag, we can review and merge it for now.
But it's not a widely used instrument, it will be removed from the Open edX repo and should be used as a plug-in in the future.

@saeedbashir
Copy link
Contributor Author

To flag, we can review and merge it for now. But it's not a widely used instrument, it will be removed from the Open edX repo and should be used as a plug-in in the future.

Merging this even for time being will be great.

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.

None yet

3 participants