Size: a a a

2017 October 02

AK

Amir Konovalov in GitFox
эх
источник

AK

Amir Konovalov in GitFox
4ка всё так же в пролёте
источник

AK

Amir Konovalov in GitFox
источник
2017 October 03

AP

Alexey Pushkarev in GitFox
ребят, добавлять заголовки через addHeader небезопасно. https://gitlab.com/terrakok/gitlab-client/blob/develop/app/src/main/java/ru/terrakok/gitlabclient/model/data/server/interceptor/AuthHeaderInterceptor.kt#L15

Я как-то упоролся в такую ошибку, использовал тоже этот метод, в итоге получилось так, что у меня задублировались заголовки. Лучше использовать request.newBuilder().header(...). В таком случае гарантируется что будет только 1 хедер с указанным ключем, а все существующие заменятся.
источник
2017 October 04

ES

Eugene Shapovalov in GitFox
@terrakok
у меня к вам вопрос по авторизации:
authInteractor.isSignedIn() вызывается в LaunchPresenter, который в свою очередь использует authData.token, т.о обращение к Preferences идёт на UI-потоке.

Корректно ли обращаться к префам на UI-потоке?
источник

KT

Konstantin Tskhovrebov in GitFox
да
источник

ES

Eugene Shapovalov in GitFox
получается, что когда мы запрашиваем signedIn state, мы точно знаем, что эта информация хранится в prefs?
источник

AP

Alexey Pushkarev in GitFox
кстати есть StrictMode где можно задать ворнинги на обращение к файлу из ui thread
источник

ES

Eugene Shapovalov in GitFox
Eugene Shapovalov
получается, что когда мы запрашиваем signedIn state, мы точно знаем, что эта информация хранится в prefs?
не соглашусь сам с собой. Логика потоков вынесена в Repository, поэтому он решает какой поток ему выбрать для определённой операции.
источник

ES

Eugene Shapovalov in GitFox
Я впервые вижу такую реализацию, обычно делают в presenter/interactor. Как вы пришли к такому решению?
источник

KT

Konstantin Tskhovrebov in GitFox
когда есть соглашение, что все данные ходят только чер UI тред, то гораздо проще управлять потоками
источник

A

Alexander in GitFox
Konstantin Tskhovrebov
когда есть соглашение, что все данные ходят только чер UI тред, то гораздо проще управлять потоками
Что-то не понял я, как это, через UI тред?
источник

KT

Konstantin Tskhovrebov in GitFox
а посмотрите на репозитории в гитфоксе
источник

A

Alexander in GitFox
Блин, хотел быстро полюбопытствовать, а теперь придётся код смотреть)
источник

ES

Eugene Shapovalov in GitFox
Konstantin Tskhovrebov
когда есть соглашение, что все данные ходят только чер UI тред, то гораздо проще управлять потоками
а что делать с составными запросами на уровне Interactor?
источник

KT

Konstantin Tskhovrebov in GitFox
Свичмап
источник

KT

Konstantin Tskhovrebov in GitFox
И комбайнить
источник

KT

Konstantin Tskhovrebov in GitFox
Все как всегда
источник

ES

Eugene Shapovalov in GitFox
получается, что так
repository.doOne.doSecond.doThird

а работа с потоками определена в Repository, то в цепи появится несколько observeOn и subscribeOn?
источник
2017 October 05

ES

Eugene Shapovalov in GitFox
я понимаю, что нам мне висит пока что Todo, но могу я начать параллельно выполнять эту
https://gitlab.com/terrakok/gitlab-client/issues/17
источник