Size: a a a

2019 October 16

IG

Ilya Gulya in GitFox
Откоментил там с предложением как почище сделать
источник

NY

Nikita Yatskivskiy in GitFox
Несколько пересмотрел PR и всё равно коммента не вижу((
источник

IG

Ilya Gulya in GitFox
Nikita Yatskivskiy
Несколько пересмотрел PR и всё равно коммента не вижу((
Упс, сорян, я finish review забыл нажать, давно гитлабом не пользовался)
источник

IG

Ilya Gulya in GitFox
Готово
источник

NY

Nikita Yatskivskiy in GitFox
А вот насчёт repeateWhen - если у нас одновременно с getAllIssueNotes при onFirstViewAttach придёт ещё и эвент, что issue обновился, то в таком случае завершатся оба запроса?
источник

NY

Nikita Yatskivskiy in GitFox
Я изначально с Disposable в поле презентера хотел как раз такого кейса избежать. Или при repeatWhen произойдёт отписка от текущего стрима и создасться новый?
источник

IG

Ilya Gulya in GitFox
Nikita Yatskivskiy
А вот насчёт repeateWhen - если у нас одновременно с getAllIssueNotes при onFirstViewAttach придёт ещё и эвент, что issue обновился, то в таком случае завершатся оба запроса?
Да, точняк, ты прав.
Тогда можно сделать как-то так:
issueChanges
   .filter { it == issueId }
   .map { Unit }
   .startWith(Unit)
   .switchMap {
     issueInteractor
         .getAllIssueNotes(projectId, issueId)
         .doOnSubscribe { viewState.showEmptyProgress(true) }
         .doAfterTerminate { viewState.showEmptyProgress(false) }
   }
источник

IG

Ilya Gulya in GitFox
Тогда лучше оставить в презентере, чтоб удобно управлять прогрессом было, как мне кажется.
источник

NY

Nikita Yatskivskiy in GitFox
Да, вполне, можно так 👍. Как раз около того в голове вариант крутился
источник

NY

Nikita Yatskivskiy in GitFox
Сегодня-завтра поправлю
источник

KT

Konstantin Tskhovrebov in GitFox
Я посмотрю завтра-послезавтра
источник
2019 October 17

NY

Nikita Yatskivskiy in GitFox
Парни, а есть ещё какие issue, например, на фикс бага, или тоже что-то типа добавления небольшой функциональности, как в случае с закрытием issue? Мне бы это попутно помогло побыстрее въехать в проект, в его структуру, в коде разобраться)
источник

NY

Nikita Yatskivskiy in GitFox
А эта задача по миграции на Moshi https://gitlab.com/terrakok/gitlab-client/issues/95 ещё актуальна?
Не понял про поддержку парсинга nullable-типов - Gson же тоже самое позволяет, разве нет?
источник

IG

Ilya Gulya in GitFox
Nikita Yatskivskiy
А эта задача по миграции на Moshi https://gitlab.com/terrakok/gitlab-client/issues/95 ещё актуальна?
Не понял про поддержку парсинга nullable-типов - Gson же тоже самое позволяет, разве нет?
Нет, gson не умеет работать с kotlin nullability.
источник

IG

Ilya Gulya in GitFox
Есть у тебя non-null поле, то gson рефлективно запишет в него null значение и у тебя упадёт при попытке обращения к полю
источник

IG

Ilya Gulya in GitFox
А moshi упадёт на этапе парсинга
источник

IG

Ilya Gulya in GitFox
Fail-fast короче
источник

NY

Nikita Yatskivskiy in GitFox
Ilya Gulya
Есть у тебя non-null поле, то gson рефлективно запишет в него null значение и у тебя упадёт при попытке обращения к полю
Типа с исключением с текстом а-ля Field specified as non-null is null?
источник

IG

Ilya Gulya in GitFox
Да
источник

NY

Nikita Yatskivskiy in GitFox
Это приоритет сейчас?
Вернее, насколько часто возникает подобная проблема с парсингом и стоит ли заняться миграцией на Moshi уже сейчас?
источник