-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Closed
Labels
Package: replayIssues related to the Sentry Replay SDKIssues related to the Sentry Replay SDK
Description
Currently, we have a quite massive index.ts file in replay, which contains a lot of logic.
We should split this up with a few key goals in mind:
- Separation of concerns - currently, this mixes e.g. the integration itself, the core class for the replay functionality, etc. all together.
- Functional programming - we try to use a more functional programming style. This does not mean we need to avoid all classes, but a decent amount of stuff can probably be extracted into atomic classes.
- Avoid very large files - this can make refactorings, understanding etc. harder
- Declare what is public & private - currently, everything is kind of public, we want to change this.
Progress
- ref(replay): Split replay integration & container class #6407
- Extract functionality from replay container class into atomic functions, where possible
- Ensure everything is correctly declared as public/private
Metadata
Metadata
Assignees
Labels
Package: replayIssues related to the Sentry Replay SDKIssues related to the Sentry Replay SDK