Skip to content

Commit

Permalink
Merge pull request #106 from Parsely/4.0-migration-guide
Browse files Browse the repository at this point in the history
4.x migration guide
  • Loading branch information
wzieba authored Feb 1, 2024
2 parents ca04c4f + 46d44e2 commit 458644c
Show file tree
Hide file tree
Showing 2 changed files with 74 additions and 0 deletions.
69 changes: 69 additions & 0 deletions MIGRATION.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,69 @@
# Parse.ly Android SDK Migration Guide

This document acts as a guide for migration, detailing every breaking change that occurs between major versions of the Parse.ly Android SDK.

## Upgrading to version 4.x

When migrating to version 4.x of the Parse.ly Android SDK, you will encounter several changes that may require updates to your codebase. The following list details these breaking changes and recommended actions:

### Removed initialization `ParselyTracker#sharedInstance` methods

Previously, the SDK was initialized `ParselyTracker#sharedInstance` method, which should be replaced with `ParselyTracker#init` method:

```kotlin
ParselyTracker.sharedInstance("example.com", 30, this, true) // before

ParselyTracker.init("example.com", 30, this, true) // now
```

Without prior initialization, the SDK will now throw `ParselyNotInitializedException` when accessing `ParselyTracker#sharedInstance`.

*Action required*: Update how the Parsely SDK is initialized and accessed as presented above.

### Removed methods or access restriction
The following methods are no longer accessible from the `ParselyTracker` class:

- `engagementIsActive`
- `flushEventQueue`
- `flushTimerIsActive`
- `getDebug`
- `getEngagementInterval`
- `getFlushInterval`
- `getVideoEngagementInterval`
- `queueSize`
- `stopFlushTimer`
- `storedEventsCount`
- `videoIsActive`

Also, `toMap` methods are no longer available for API consumers:

- `ParselyMetadata#toMap`
- `ParselyVideoMetadata#toMap`

*Action required*: Those methods aren't needed to use the SDK. Remove any code segments that interact with them.

### Restricted access to internal API connection
`ParselyAPIConnection`, which might have been previously used for direct API interactions, is no longer accessible to the API consumer.

*Action required*: Remove any code segments that interact with `ParselyAPIConnection` as this class is now an implementation detail of the SDK.

### Non-nullable `urlRef` parameters
Parameters that were previously nullable when calling various tracking methods are now non-nullable.

- In the `ParselyTracker` class, the `urlRef` parameter in the following methods requires a string value:
- `trackPageview`
- `startEngagement`
- `trackPlay`

*Action required*: When calling these methods, ensure you provide a valid string. If there is no value available, pass an empty string `""` instead of `null`.

### Debug configuration change
Configuring the debug mode during the SDK initialization has been simplified to a "dry run" mode flag.

- The method `ParselyTracker#setDebug` is no longer available.

*Action required*: To enable `debug` mode, when initializing the `ParselyTracker`, set the `dryRun` property to `true`.

---

For clarification on any of these changes or assistance with migrating your code to the latest version of the Parse.ly Android SDK, please contact our support team at `[email protected]` or open a GitHub issue.
5 changes: 5 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,3 +18,8 @@ implementation("com.parsely:parsely:<release_version>")

Full instructions and documentation can be found on
the [Parse.ly help page](https://docs.parse.ly/android-sdk/).

## Migration to 4.0.0

Version 4.0.0 of the SDK introduces significant updates and breaking changes that enhance performance and add new features.
These changes may require modifications to your existing code. For detailed instructions on how to adapt your code to these changes, please refer to our [Migration Guide](MIGRATION.md).

0 comments on commit 458644c

Please sign in to comment.