Size: a a a

2017 October 17

KT

Konstantin Tskhovrebov in GitFox
по каждому новому экрану надо завести задачу
источник

KT

Konstantin Tskhovrebov in GitFox
а в описании и обсуждении будем заводить список фич в первой версии
источник
2017 October 19

NG

Nikita Gusarov in GitFox
@terrakok ты выше сообщил, мол "дизайнер смог нарисовать.". Сможешь передать макеты скетча которые я подготовил для страницы которую я реализую? Возможно ему пригодиться для изменения, учета
источник

KT

Konstantin Tskhovrebov in GitFox
надо закрепить в задаче
источник

ES

Eugene Shapovalov in GitFox
@terrakok
они там прикреплены (скриншоты некоторые) и уже есть mr.
источник

KT

Konstantin Tskhovrebov in GitFox
планирую вечером разобраться
источник

NG

Nikita Gusarov in GitFox
Konstantin Tskhovrebov
надо закрепить в задаче
я бы прикрепил, да максимальный размер файла для прикрепления - 10мб. Просто добавлю ссылку на публичный файл в гуглдоке
источник

KT

Konstantin Tskhovrebov in GitFox
@abbath0767
увидел я ваш МР и приуныл..

нельзя делать такие МР. столько кода никто прочитать за раз не сможет.

посмотрите пожалуйста на этапы любого экрана, который делали ребята до вас.
все начинать надо с дата слоя (серверные модельки и АПИ). И создать МР в ветку фичи.

После того как этот этап будет принят, то переходить к реализации репозитория...

и так далее
источник

KT

Konstantin Tskhovrebov in GitFox
это позволяет:
1) делать первые этапы совершенно не думаю о UI
2) тщательно проводить код ревью
3) не допускать ошибок
4) идти от простого к сложному, постепенно синхронизируя стиль кода
источник

AP

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

KT

Konstantin Tskhovrebov in GitFox
это не значит, что надо переписать. просто вынесите все в отдельные ветки и присылайте поочереди😉
источник

KT

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

AP

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

DS

Denis S in GitFox
Alexey Pushkarev
ну то есть не знаю как Вы, но у меня не получается прям сразу учесть все. Если у вас так получается, расскажите как Вам удается?
вначале можно сделать архитектурное ревью, типо написать вначале какие будут энтити, накидать интерфейсы которые будут, описать возможные флоу на экране, убедиться что апи возращает все нужное, раскидать по слоям. да, редко когда удается прям все возможные стейты учесть, но если хорошо спланировать, и общаться с продуктовой командой и дизайнером, и учесть все стейты заранее, то потом это окупиться сильно, чтобы сократтиь переделки. потому что в процессе обсуждения и планирование может выяснится что фича не нужна, или есть какой то скрытый стейт и переход, который надо заложить. плюс если у тебя будет код отделен на слои и будут тесты, ты сможешь изи рефачить/добавлять/удалять, особо не переживая. это кэпство конечно, но на практике работает, особенно если проект не имеет большой кодобазы/легаси
источник
2017 October 20

NG

Nikita Gusarov in GitFox
Konstantin Tskhovrebov
@abbath0767
увидел я ваш МР и приуныл..

нельзя делать такие МР. столько кода никто прочитать за раз не сможет.

посмотрите пожалуйста на этапы любого экрана, который делали ребята до вас.
все начинать надо с дата слоя (серверные модельки и АПИ). И создать МР в ветку фичи.

После того как этот этап будет принят, то переходить к реализации репозитория...

и так далее
Хорошо, если считаете что так будет лучше то разобью на ветки и будем постепенно кидать mr.
источник
2017 October 22

KT

Konstantin Tskhovrebov in GitFox
Сегодня завел все экраны в задачи дизайнеру. Когда нарисует, то будут пошаговые задачи в гитлабе для реализации
источник
2017 October 26

VP

Vladimir Parfenov in GitFox
Eugene Привет. Экран TODO доделывать планируешь? (все вроме дизайна)
источник

ES

Eugene Shapovalov in GitFox
Да. На этих выходных займусь им. Не мог уделять время.
источник

VP

Vladimir Parfenov in GitFox
Ок
источник
2017 October 29

ES

Eugene Shapovalov in GitFox
@terrakok
я теперь могу выкачать оригинальный проект и использовать бранч в этом проекте, а не в форкнутом?
источник