Size: a a a

2019 October 17

ES

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

ES

Eugene Shapovalov in GitFox
если менять на fail-fast, то нужно быть увереным, что каждый кусочек приложения регрессионно протестирован.
источник

IG

Ilya Gulya in GitFox
@NikiJava Там есть ещё 2 фичи помеченные как "hot", мб их сделать? Они схожи с той что ты уже запилил.
источник

IG

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

NY

Nikita Yatskivskiy in GitFox
Ilya Gulya
@NikiJava Там есть ещё 2 фичи помеченные как "hot", мб их сделать? Они схожи с той что ты уже запилил.
Вот, как раз на них тоже обратил сейчас внимание
источник

NY

Nikita Yatskivskiy in GitFox
Могу да, ими заняться
источник

NY

Nikita Yatskivskiy in GitFox
после того, как таску с close issue замерджим тогда
источник

KT

Konstantin Tskhovrebov in GitFox
Nikita Yatskivskiy
Вот, как раз на них тоже обратил сейчас внимание
👍 еще не добрался, но помню про вас)
источник

NY

Nikita Yatskivskiy in GitFox
Konstantin Tskhovrebov
👍 еще не добрался, но помню про вас)
Ок, жду ревью)
источник

NY

Nikita Yatskivskiy in GitFox
Ilya Gulya
Да, точняк, ты прав.
Тогда можно сделать как-то так:
issueChanges
   .filter { it == issueId }
   .map { Unit }
   .startWith(Unit)
   .switchMap {
     issueInteractor
         .getAllIssueNotes(projectId, issueId)
         .doOnSubscribe { viewState.showEmptyProgress(true) }
         .doAfterTerminate { viewState.showEmptyProgress(false) }
   }
я там поправил rx-цепочки в PR - выпилил дополнительные Disposable, заюзал switchMap, но без маппинга на Unit - подумал, что можно тот же issueId заюзать, который в презентере уже есть, и добавлять его в startWith.
источник

IG

Ilya Gulya in GitFox
Nikita Yatskivskiy
я там поправил rx-цепочки в PR - выпилил дополнительные Disposable, заюзал switchMap, но без маппинга на Unit - подумал, что можно тот же issueId заюзать, который в презентере уже есть, и добавлять его в startWith.
о, тоже вариант, да
источник

IG

Ilya Gulya in GitFox
Nikita Yatskivskiy
я там поправил rx-цепочки в PR - выпилил дополнительные Disposable, заюзал switchMap, но без маппинга на Unit - подумал, что можно тот же issueId заюзать, который в презентере уже есть, и добавлять его в startWith.
Думаю стоит ловить ошибки внутри switchMapSingle. Иначе вся подписка на issueChanges сломается при ошибке, а не только один запрос.
источник

IG

Ilya Gulya in GitFox
Nikita Yatskivskiy
я там поправил rx-цепочки в PR - выпилил дополнительные Disposable, заюзал switchMap, но без маппинга на Unit - подумал, что можно тот же issueId заюзать, который в презентере уже есть, и добавлять его в startWith.
Типа
.switchMapMaybe {
 getIssue()
     .toMaybe()
     .doOnSuccess { (issue, mdDescription) -> viewState.showDetails(issue, mdDescription) }
     .doOnError { errorHandler.proceed(it, { viewState.showMessage(it) }) }
     .onErrorComplete()
}
источник

NY

Nikita Yatskivskiy in GitFox
Да, пропустил этот момент. Сейчас поправлю)
источник

NY

Nikita Yatskivskiy in GitFox
@ilyagulya а имеет же смысл в таком случае оставить обработку onError и в subscribe тоже? С одной стороны - под капотом у issueChanges релейка, у которой onError нет, а с другой стороны - мы работаем с ней через интерфейс Observable и поэтому onError чисто теоретически возможен.
источник

IG

Ilya Gulya in GitFox
Nikita Yatskivskiy
@ilyagulya а имеет же смысл в таком случае оставить обработку onError и в subscribe тоже? С одной стороны - под капотом у issueChanges релейка, у которой onError нет, а с другой стороны - мы работаем с ней через интерфейс Observable и поэтому onError чисто теоретически возможен.
Не уверен)
Я обычно не оставляю если на текущий момент on Error не может произойти)
источник

NY

Nikita Yatskivskiy in GitFox
@ilyagulya поправил, перепроверил - всё норм)
источник

IG

Ilya Gulya in GitFox
Nikita Yatskivskiy
@ilyagulya поправил, перепроверил - всё норм)
Каеф
источник

IG

Ilya Gulya in GitFox
Ждём Костю)
источник

IG

Ilya Gulya in GitFox
Nikita Yatskivskiy
@ilyagulya поправил, перепроверил - всё норм)
Может ты хочешь продолжить мою задачу про markdown после тех двух тасок?)
источник