refactor: Amplitude upgrade #31
Open
+2,551
−2,589
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This commit refactors core SDK files for improved null safety, device info caching, and session management. It introduces bulk event logging, updates event buffer logic for concurrency, and adds resource cleanup via dispose methods. Test coverage is expanded with new and updated tests, and analysis/linter rules are streamlined. The changes improve reliability, performance, and maintainability of the SDK.
Fixed issues that were disabling executing the example app.
Optimizes event processing by pre-fetching device info and session IDs, reducing redundant asynchronous calls. Disables advertising ID retrieval due to instability issues. Improves event buffering by scheduling flush timer only when needed and optimizing event timestamping. Uses a single UUID generator instance to improve performance.
Reduce number of async awaits for db init.
Improves the performance of event posting by using isolate computation for large payloads. This change introduces asynchronous posting and resource cleanup. It also addresses potential timeout issues with a configurable timeout duration and error handling.
Reorganized source files into feature-based directories, introduced ObjectBox for data storage, and updated exports in amplitude_flutter.dart. Removed obsolete files and tests, added new models and storage implementations, and migrated existing code to the new structure for improved maintainability.
Added missing comments on classes/functions/variables
Crdzbird
added
documentation
Introduced new public getters: 'properties' in Event and Metric, and 'data' in Log, to provide direct access to their respective internal maps. This improves encapsulation and allows external code to access these fields more explicitly.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
DETAILS
SqliteandObjectBoxbased on the implementations.*** IMPORTANT ***
Since this module was already migrated into the main project, this PR is for the sake of keeping track of the new changes with more order.