Delete :stream-chat-android-ui-utils module #6080
Merged
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.


🎯 Goal
Moves the extensions from the
:stream-chat-android-ui-utilsmodule to the:stream-chat-android-ui-commonmodule, and removes the now empty module.🛠 Implementation details
Moves all utility methods from the
ui-utilsmodule to theui-common:io.getstream.chat.android.uiutils.model.MimeType->io.getstream.chat.android.ui.common.model.MimeTypeio.getstream.chat.android.uiutils.extension->io.getstream.chat.android.ui.common.utils.extensionspackageio.getstream.chat.android.uiutils.util.EmojiUtil->io.getstream.chat.android.ui.common.utils.EmojiUtilAdditionally, took the liberty to hide several methods from the public API:
Date.isInLastMinuteis nowprivateString.containsLinksis nowinternalString.addSchemeToUrlIfNeededis now marked as@InternalStreamChatApiContext.openSystemSettingsis now marked as@InternalStreamChatApiadjustColorBrightnessis now marked as@InternalStreamChatApiAll of there methods are extensions on non-Stream classes and pollute the public namespace. Since they are also internal logic, I don't see a reason to expose them publicly. (If you think we should keep them public, I will be happy the revert this change)
Added some missing tests.
Deletes the module and removes its usage.
🎨 UI Changes
NA
🧪 Testing
Not much to test, now customers don't need to add the
ui-utilsmodule, they can access the same methods from the UI SDKs.