Upgrade Stream Chat Depencency to v6 #34
Conversation
| stream-state = { module = "io.getstream:stream-chat-android-state", version.ref = "stream" } | ||
| stream-offline = { module = "io.getstream:stream-chat-android-offline", version.ref = "stream" } |
There was a problem hiding this comment.
Offline artifact has been split into:
stream-chat-android-offline -> Responsible for data persistence on the device
stream-chat-android-state -> Responsible to provide reactive data
| stream-offline = { module = "io.getstream:stream-chat-android-offline", version.ref = "stream" } | ||
| stream-ui-components = { module = "io.getstream:stream-chat-android-ui-components", version.ref = "stream" } | ||
| stream-firebase = { module = "io.getstream:stream-chat-android-pushprovider-firebase", version.ref = "stream" } | ||
| stream-firebase = { module = "io.getstream:stream-android-push-firebase", version.ref = "streamPush" } |
There was a problem hiding this comment.
Push artifact has been extracted from the principal repository, but the implementation is the same
| import android.content.Context | ||
| import android.content.Intent | ||
| import androidx.startup.Initializer | ||
| import io.getstream.android.push.firebase.FirebasePushDeviceGenerator |
There was a problem hiding this comment.
"Push Classes" are into a new artifact with different package name
| config = StatePluginConfig( | ||
| backgroundSyncEnabled = true, | ||
| userPresence = true, | ||
| persistenceEnabled = true, |
There was a problem hiding this comment.
Whenever the Offline Plugin is used, the persistence is enabled. This flag is not needed anymore
| backgroundSyncEnabled = true, | ||
| userPresence = true, | ||
| persistenceEnabled = true, | ||
| uploadAttachmentsNetworkType = UploadAttachmentsNetworkType.NOT_ROAMING |
There was a problem hiding this comment.
This configuration is added to the ChatClient, it is not a configuration of the offline plugin anymore.
| ChatClient.Builder(context.getString(R.string.stream_api_key), context) | ||
| .notifications(createNotificationConfig(), createNotificationHandler(context)) | ||
| .withPlugin(offlinePluginFactory) | ||
| .withPlugins(offlinePluginFactory, statePluginFactory) |
There was a problem hiding this comment.
Both plugins are included
|
|
||
| import androidx.annotation.WorkerThread | ||
| import io.getstream.chat.android.client.models.User | ||
| import io.getstream.chat.android.models.User |
There was a problem hiding this comment.
Models has been moved to a different package
| result.onSuccessSuspend { | ||
| emit(result.data()) | ||
| emit(it) |
There was a problem hiding this comment.
Result class has been improved and extracted to a different artifact.
It improves how it is used taking advantage of Kotlin language.
| android:layout_height="match_parent"> | ||
|
|
||
| <io.getstream.chat.android.ui.channel.list.header.ChannelListHeaderView | ||
| <io.getstream.chat.android.ui.feature.channels.header.ChannelListHeaderView |
There was a problem hiding this comment.
Some Views have been moved to other package names to keep the parity with the compose artifact
| /> | ||
|
|
||
| <io.getstream.chat.android.ui.message.input.MessageInputView | ||
| <io.getstream.chat.android.ui.feature.messages.composer.MessageComposerView |
There was a problem hiding this comment.
This class has been renamed to keep the parity with the compose artifact
JcMinarro
force-pushed
the
upgrade-stream-v6
branch
from
d9cd625 to
be6416b
Compare
JcMinarro
force-pushed
the
upgrade-stream-v6
branch
from
be6416b to
ed3f183
Compare
|
Do not merge this PR yet. |
Upgrade App to the new Stream Chat V6