Size: a a a

2017 November 08

VP

Vladimir Parfenov in GitFox
Allowingstateloss спасает. Хотя выглядит как костыль
источник

AK

Aleksei Korshun in GitFox
Vladimir Parfenov
Allowingstateloss спасает. Хотя выглядит как костыль
это и есть костыль 🙂
источник

AK

Aleksei Korshun in GitFox
проблема выше 🙂
источник

AK

Aleksei Korshun in GitFox
@terrakok а багфикс MR создавать в девелоп?
источник

KT

Konstantin Tskhovrebov in GitFox
А куда ещё? Главное ветку правильно назовите
источник

AK

Aleksei Korshun in GitFox
bugfix указал
источник

KT

Konstantin Tskhovrebov in GitFox
fix/some_bug
источник

KT

Konstantin Tskhovrebov in GitFox
Это описано уже
источник

AK

Aleksei Korshun in GitFox
да, по привычке написал bugfix
источник

AK

Aleksei Korshun in GitFox
сейчас поправлю
источник
2017 November 09

KT

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

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

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

я сделаю обработчик общим и буду фильтровать одновременные одинаковые вызовы. либо как-то иначе. еще подумаю над этим
источник

AK

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

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

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

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

KT

Konstantin Tskhovrebov in GitFox
вот и пища для размышлений. проблема чуть глубже
источник

KT

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

KT

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

AK

Aleksei Korshun in GitFox
а можно в authactivity скипать команду на авторизацию?
источник

AK

Aleksei Korshun in GitFox
с точки зрения архитектуры не уверен что верно, но смысл в том что если открыто окна авторизации и роутер говорит что надо показать авторизацию, то мы игнорируем эту команду там, ввиду того что мы уже на этом экране?
источник

KT

Konstantin Tskhovrebov in GitFox
нет - это костыль
источник

AK

Aleksei Korshun in GitFox
Konstantin Tskhovrebov
нет - это костыль
А можно более подробно почему?)))
источник

KT

Konstantin Tskhovrebov in GitFox
костыль = не решение проблемы, и даже не решение какого-то последствия, а неявное поведение, которое скрывает проблему
источник