Skip to content

Telegramflow fix#2

Open
matzuk wants to merge 2 commits intotdlibx:masterfrom
matzuk:telegramflow_fix
Open

Telegramflow fix#2
matzuk wants to merge 2 commits intotdlibx:masterfrom
matzuk:telegramflow_fix

Conversation

@matzuk
Copy link
Copy Markdown

@matzuk matzuk commented Sep 10, 2021

Changes:

  1. I was forced to update the Kotlin Gradle plugin to work with a new version of Kotlin flows. That's why I removed a code about Dokka and Publication temporary (honestly, You didn't publish this library, that's why I hope it's not critical). Also, I raised versions of other libraries.
  2. Refactoring of TelegramFlow. I used callbackFlow and transformed it into SharedFlow with a scope = CoroutineScope(SupervisorJob() + Dispatchers.IO). Also, I wrapped sendFunctionAsync into context = Dispatchers.IO.
  3. Any very small changes. The most important is a user can get flows from TelegramFlow only after the calling of attachClient method.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant