Size: a a a

2017 November 09

ES

Eugene Shapovalov in GitFox
А нельзя добавить поведение типа onNewIntent?
источник

KT

Konstantin Tskhovrebov in GitFox
Это все к истории костылей
источник

ES

Eugene Shapovalov in GitFox
Konstantin Tskhovrebov
как быстрый вариант в голову приходит при явной проверке тоже кидать ошибку в обработчик ошибок
так получается, что ошибка будет типа AuthSessionIsAlreadyUpdatingException и если выпадает такая ошибка, то ничего делать не требуется?
источник

AK

Aleksei Korshun in GitFox
Konstantin Tskhovrebov
я вам ответил, но повторю здесь

вы устранили последствие, но не причину проблемы.
проблема действительно в том, что два разных презентера одновременно получают ошибку сессии и пытаются одновременно кинуть на логин скрин.

это может произойти где угодно, поэтому решать надо саму проблему, а не частные проявления

я сделаю обработчик общим и буду фильтровать одновременные одинаковые вызовы. либо как-то иначе. еще подумаю над этим
А как обработчик поймет что уже не надо фильтровать? Ему надо знать о состоянии навигации получается, что не есть хорошо. Мне все же кажется, что это ответственность навигатора, как выполнить команду роутера.
источник

AP

Alexey Pushkarev in GitFox
ребят, мб оффтоп, но всё же спрошу. А как вы относитесь к куче мелких коммитов в ветке? То есть обязательно ли поддерживать порядок в коммитах? Если я делаю коммит потому что я сделал какие-то изменения с одного компа и хочу продолжить дома.
источник

AP

Alexey Pushkarev in GitFox
и всякие заморочки со сквошем коммитов в 1 при пуше, нужно ли тратить время на такие заморочки? Вы здесь такое практикуете?
источник

AK

Aleksei Korshun in GitFox
Aleksei Korshun
А как обработчик поймет что уже не надо фильтровать? Ему надо знать о состоянии навигации получается, что не есть хорошо. Мне все же кажется, что это ответственность навигатора, как выполнить команду роутера.
Я понял почему так не надо делать, правильным, кажется отмена всех действующих запросов, при получении первой unauthorize
источник

KT

Konstantin Tskhovrebov in GitFox
Aleksei Korshun
Я понял почему так не надо делать, правильным, кажется отмена всех действующих запросов, при получении первой unauthorize
Достаточно просто фильтровать ошибки, которые одинаковые и произошли практически одновременно (50 мс)
источник

AK

Aleksei Korshun in GitFox
в условиях плохого интерета вырасти может такой интервал до 3с
источник

AK

Aleksei Korshun in GitFox
тогда на экране произойдет реплейс
источник

KT

Konstantin Tskhovrebov in GitFox
Alexey Pushkarev
и всякие заморочки со сквошем коммитов в 1 при пуше, нужно ли тратить время на такие заморочки? Вы здесь такое практикуете?
Коммиты должны отображать то, как вы работали. Это не красивая история, а реальная. Часть на работе и часть дома - значит т будет промежуточный коммит
источник

AP

Alexey Pushkarev in GitFox
Konstantin Tskhovrebov
Коммиты должны отображать то, как вы работали. Это не красивая история, а реальная. Часть на работе и часть дома - значит т будет промежуточный коммит
возможно для отчетности перед работодателем это и имеет смысл, а в опенсорсе зачем?
источник

KT

Konstantin Tskhovrebov in GitFox
Причем здесь работодатель? При чем опенсорс? Это гит. Все. Остальное не сюда.
источник

ES

Eugene Shapovalov in GitFox
Konstantin Tskhovrebov
Достаточно просто фильтровать ошибки, которые одинаковые и произошли практически одновременно (50 мс)
а это безопасно полагаться на определённое время для фильтрации?
источник

AK

Aleksei Korshun in GitFox
тогда на экране произойдет реплейс
источник

AK

Aleksei Korshun in GitFox
Мне тоже кажется это не оч
источник

KT

Konstantin Tskhovrebov in GitFox
Тут важен контекст. Если происходит ОДНА И ТА ЖЕ ошибка позже, то значит ее можно обработать повторно.
Не надо уходить в абстракции.
Мы же ставим дебаунс на нажатие кнопки, и не паримся, что "а вдруг юзер реально хотел дважды нажать"
источник

KT

Konstantin Tskhovrebov in GitFox
Эта задержка не из головы, а из предметной области.
источник

KT

Konstantin Tskhovrebov in GitFox
Это очень важное отличие
источник

VP

Vladimir Parfenov in GitFox
В code style добавлена информация о структуре класса: https://gitlab.com/terrakok/gitlab-client/wikis/code_style.md
источник