Size: a a a

2017 September 21

KT

Konstantin Tskhovrebov in GitFox
Eugene Shapovalov
@terrakok
Хотел узнать у вас такой момент:
Когда пользователь проскроливает до новой страницы, то view говорит presenter loadNewPage, т.о нарушается контракт слабой view и presenter.
Если это послабление осознанное, то с чем это связано?
Вы о чем? Это обычное действие! Как клик на кнопку. Только событие, что достигли конца списка
источник

KT

Konstantin Tskhovrebov in GitFox
Vitalii Dmitriev
@terrakok
А какая лицензия у проекта?
Может, я не туда смотрю, но ни файла лицензии в корне, ни упоминания — ничего. Может выбрать тогда? Я и в абаут её напишу, и хедеры можно добавлять будет.
На всякий случай завёл задачу. Предлагаю использовать Apache 2.0.
Никакой. Как видите, у проекта нет ни лицензии, ни контрибьюшн гайда, ни вики.
Ещё не набралась команда достаточная, чтобы можно было делать это все. Сейчас основная цель - это собрать список задач для первого релиза. Нарисовать для них дизайн, и выпустить приложение ДО нового года. После этого можно будет и статей написать, и ридми оформить и народа больше заинтересовать.
Сейчас над проектом работает человек пять от силы, а я один все это дело проверяю и веду дальше.
источник

KT

Konstantin Tskhovrebov in GitFox
Как вы понимаете, это кроме выступлений, основной работы и всего прочего
источник

VD

Vitalii Dmitriev in GitFox
Понял.
источник

ES

Eugene Shapovalov in GitFox
Обычно view говорит presenter, что событие свершилось (buttonClicked),а в нашем случае loadNewPage.
источник

ES

Eugene Shapovalov in GitFox
Вы о чем? Это обычное действие! Как клик на кнопку. Только событие, что достигли конца списка
источник

AK

Aleksei Korshun in GitFox
ну можно переименовать метод, onBottomEnd например но это будет не так читаемо наверное
источник

ES

Eugene Shapovalov in GitFox
И получается, что нарушается контракт слабой view и presenter
источник

AK

Aleksei Korshun in GitFox
в чем?
источник

ES

Eugene Shapovalov in GitFox
View должна говорить о событии, а не то, что нужно сделать.
источник

AK

Aleksei Korshun in GitFox
переименуйте метод
источник

AK

Aleksei Korshun in GitFox
суть не в названиях
источник

KT

Konstantin Tskhovrebov in GitFox
Метод должен говорить за себя.
источник

ES

Eugene Shapovalov in GitFox
onNewPageScrolled(...)
источник

ES

Eugene Shapovalov in GitFox
@terrakok
я сейчас на стадии Interactor и просматривая Issues, я не нашёл для него unit-test, необходимо ли для todos его писать?
источник
2017 September 22

ES

Eugene Shapovalov in GitFox
ну я всё-таки написал его и уже создал mr
источник

VD

Vitalii Dmitriev in GitFox
Подскажите, что я делаю не так?
Закоммитил изменения, запушил себе на ветку, делаю MR.
Пишет, что автоматический мёрдж невозможен потому что отсутствует source branch.
источник

ES

Eugene Shapovalov in GitFox
Vitalii Dmitriev
Подскажите, что я делаю не так?
Закоммитил изменения, запушил себе на ветку, делаю MR.
Пишет, что автоматический мёрдж невозможен потому что отсутствует source branch.
а Константин создал уже ветку в своём проекте?
источник

VD

Vitalii Dmitriev in GitFox
А, в этом проблема? Не знаю.
Тогда совсем не понимаю, как тожне процесс MR проходить.
источник

ES

Eugene Shapovalov in GitFox
пока создавайте mr на develop, когда он аппрувнет первый, то он создат идентичную ветку в original project, и вы поменяете target_branch
источник