Запросы в сеть с Clean Architecture и MVVM. Boilerplate ч. 2

А насчет internal top функции согласен с вами можно и так вынести, но просто изначально в коде ещё есть такой же метод для запроса с пагинацией. Просто не видел смысла выносить.

Для себя выучил, что наследование - последнее, к чему надо прибегать.

Хотелось бы услышать от вас как бы вы реализовали эту логику?

class SignInRepositoryImpl @Inject constructor(
    private val service: SignInApiService,
    private val userData: UserData
) : BaseRepository(), SignInRepository {

    override fun signIn(userSignIn: UserSignIn) = doRequest {
        service.signIn(userSignIn.fromDomain()).toDomain()
        	.also { userData.saveToken(it.token) }
    }
}

Запросы в сеть с Clean Architecture и MVVM. Boilerplate ч. 2

Согласен с комментарием о doSomethingInSuccess, калбэк видится лишним, ведь если запрос упадёт, выпадет исключение и выполнение функции прекратится.

От себя ещё замечу, что наследование BaseRepository так же можно заменить на internal top level функцию или композицию

Адаптация Android-приложения для незрячих

Давно не читал ничего настолько интересного, спасибо

GitFlic. Российский GitHub. Рассмотрение сервиса и его нюансы

Достойный аналог на ₽2,1 млрд, ничего не скажешь.