Size: a a a

2017 September 18

ES

Eugene Shapovalov in GitFox
всегда бесило в Java конструктарах : одно дело указывать nullable, но и одновременно приходилось дополнительно указывать non-null, ведь если не указать, то тогда можно спутать.

Но допустим мы передаем в конструкто Context, Interactor и т.д. Нужны ли им аннотации? Кто додумается передавать null context...
источник

KT

Konstantin Tskhovrebov in GitFox
уходим от темы....
мне надо добить авторизацию на кастомных серверах
источник

ES

Eugene Shapovalov in GitFox
спасибо большое за разъяснение, я завтра переделаю mr.
источник

VD

Vitalii Dmitriev in GitFox
Konstantin Tskhovrebov
@ilyagulya я вижу, вы хотите взять фичу с авторизацией, но там много сложных архитектурных вещей надо сделать. я на этой неделе собираюсь сам ее сделать.
давайте чтобы не пересекаться отставим ее мне. боюсь, что мне многое надо будет объянять
Если нет интернета, то второе после сплеша, что показывает приложение — экран webview с ошибкой no internet connection.
Хотел создать задачу на это: создавать или она будет зарезолвлена во время разработки фичи с авторизацией?
источник

KT

Konstantin Tskhovrebov in GitFox
Создавать. Лучше лишняя задача, чем ее отсутствие
источник

KT

Konstantin Tskhovrebov in GitFox
@eugenetereshkov
осталось 5 совсем мелких правок.
а еще желательно прикрепить скрин того, что получилось. Прямо там в комментах
источник

ES

Eugene Shapovalov in GitFox
@terrakok
сейчас смотрю API events, т.к там присутствует логика Target.
Т.е парсим мы enums с большой буквоой, а отправляем с маленькой 🙂
источник

KT

Konstantin Tskhovrebov in GitFox
скорее всего это значит, что будут нужны разные enum'ы
источник

KT

Konstantin Tskhovrebov in GitFox
я не смотрел туда еще
источник

ES

Eugene Shapovalov in GitFox
Konstantin Tskhovrebov
скорее всего это значит, что будут нужны разные enum'ы
Они везде используют Target, но не описали в одном месте, все его типы 🙁
источник

ES

Eugene Shapovalov in GitFox
Константин, я заимплементил требуемые изменения. Только мои изменения будут затрагивать то, что делал @ilyagulya.
источник

KT

Konstantin Tskhovrebov in GitFox
потом будете конфликты разрешать. сейчас не надо думать об этом
источник

ES

Eugene Shapovalov in GitFox
окей
источник
2017 September 19

ES

Eugene Shapovalov in GitFox
@terrakok
я заимплементил последние замечания + изменил ветку + разрешил конфликт.
источник

KT

Konstantin Tskhovrebov in GitFox
если ветка была изменена, то конфликта быть не должно
источник

ES

Eugene Shapovalov in GitFox
я предполагаю, что вы ветку создали на основе develop, а я не мержил изменения из него в свою ветку (раньше там был класс IssueState, но я его удалил и создал TargetState).
источник

KT

Konstantin Tskhovrebov in GitFox
поэтому не нужно ничего тянуть в ветку до завершения фичи, после этого уже втянуть все свежие изменения
источник

KT

Konstantin Tskhovrebov in GitFox
короче я посмотрю вечером
источник

ES

Eugene Shapovalov in GitFox
я только могу предположить, но вы ветку в origin project создали на основе develop, который отличался от того, на основе которого я начинал реализовывать фичу
источник

KT

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