ES
и по такому плану реализуем функционал
Size: a a a
2018 October 27
ES
также можно взять эту таску
ES
2018 October 28
KT
@ilyagulya я бегло посмотрел мр, там есть архитектурные моменты, которые мне надо переварить. интерактор в маркдаун конвертере, который грузит лейблы...
надо это обдумать.
надо это обдумать.
IG
@ilyagulya я бегло посмотрел мр, там есть архитектурные моменты, которые мне надо переварить. интерактор в маркдаун конвертере, который грузит лейблы...
надо это обдумать.
надо это обдумать.
Только не в маркдаун конвертере, а в провайдере маркдаун конвертера)
Специализированном, чтоб не дублировать код в презентерах.
Специализированном, чтоб не дублировать код в презентерах.
IG
Лучше я ничего не придумал, к сожалению. Навыков архитектурных не так много.
KT
Лучше я ничего не придумал, к сожалению. Навыков архитектурных не так много.
продумал. план такой: надо сделать кастомную вью MarkdownTextView, у которой будет свой презентер.
Это хорошо описано в мокси вики. делается просто. мы часто используем.
у этой вью будет метод setMarkdown(text: String, projectId: Long? = null)
она будет передавать все это дело своему презентеру, а тот будет грузить лейблы, отдвать на парсинг марквону и тд
важно: это то что можно засетить новый текст, до того как распарсился старый (в ресайклере). поэтому надо учесть отписку и запуск нового процесса и тд
это хорошее архитектурное задание 😊 готовы взять?
если да, то надо отдельным МР создать это в проекте (пока без лейблов, а как отдельную задачу)
в MarkdownPresenter будет инжектиться ProjectInteractor и MarkdownConverter
главное все правильно и аккуартно приготовить))
Это хорошо описано в мокси вики. делается просто. мы часто используем.
у этой вью будет метод setMarkdown(text: String, projectId: Long? = null)
она будет передавать все это дело своему презентеру, а тот будет грузить лейблы, отдвать на парсинг марквону и тд
важно: это то что можно засетить новый текст, до того как распарсился старый (в ресайклере). поэтому надо учесть отписку и запуск нового процесса и тд
это хорошее архитектурное задание 😊 готовы взять?
если да, то надо отдельным МР создать это в проекте (пока без лейблов, а как отдельную задачу)
в MarkdownPresenter будет инжектиться ProjectInteractor и MarkdownConverter
главное все правильно и аккуартно приготовить))
IG
продумал. план такой: надо сделать кастомную вью MarkdownTextView, у которой будет свой презентер.
Это хорошо описано в мокси вики. делается просто. мы часто используем.
у этой вью будет метод setMarkdown(text: String, projectId: Long? = null)
она будет передавать все это дело своему презентеру, а тот будет грузить лейблы, отдвать на парсинг марквону и тд
важно: это то что можно засетить новый текст, до того как распарсился старый (в ресайклере). поэтому надо учесть отписку и запуск нового процесса и тд
это хорошее архитектурное задание 😊 готовы взять?
если да, то надо отдельным МР создать это в проекте (пока без лейблов, а как отдельную задачу)
в MarkdownPresenter будет инжектиться ProjectInteractor и MarkdownConverter
главное все правильно и аккуартно приготовить))
Это хорошо описано в мокси вики. делается просто. мы часто используем.
у этой вью будет метод setMarkdown(text: String, projectId: Long? = null)
она будет передавать все это дело своему презентеру, а тот будет грузить лейблы, отдвать на парсинг марквону и тд
важно: это то что можно засетить новый текст, до того как распарсился старый (в ресайклере). поэтому надо учесть отписку и запуск нового процесса и тд
это хорошее архитектурное задание 😊 готовы взять?
если да, то надо отдельным МР создать это в проекте (пока без лейблов, а как отдельную задачу)
в MarkdownPresenter будет инжектиться ProjectInteractor и MarkdownConverter
главное все правильно и аккуартно приготовить))
попробую
KT
еще из простых задач для начинающих работяг: убрать из приложения список разработчиков и сделать кнопку, которая будет открывать https://gitlab.com/terrakok/gitlab-client/graphs/develop
ES
вот issue !
KT
открывать просто во внешнем браузере
KT
вот issue !
2018 October 29
ES
работяги, ожидайте 1.3.0 из маркета:
🔥 Added merge requrest change list
🔧 Fixed bug with loading private project image
🔧 Fixed bug with restoring main tab
🔥 Added merge requrest change list
🔧 Fixed bug with loading private project image
🔧 Fixed bug with restoring main tab
KT
теперь клиент несет не просто ознакомительную роль! можно смотреть дифф мерж реквеста и писать комментарии в его обсуждении!
ES
Работяги, для желающих влиться в проект есть небольние таски:
1) https://gitlab.com/terrakok/gitlab-client/issues/104
2) https://gitlab.com/terrakok/gitlab-client/issues/102
3) https://gitlab.com/terrakok/gitlab-client/issues/101
4) https://gitlab.com/terrakok/gitlab-client/issues/76
Идеально подходящие таски для начала.
1) https://gitlab.com/terrakok/gitlab-client/issues/104
2) https://gitlab.com/terrakok/gitlab-client/issues/102
3) https://gitlab.com/terrakok/gitlab-client/issues/101
4) https://gitlab.com/terrakok/gitlab-client/issues/76
Идеально подходящие таски для начала.
ES
2018 October 30
KT
@m_myalkin ваш фикс улетел в маркет)
ES
@m_myalkin
привет ещё раз, насчёт mr. Я для вас создал базовый mr, который состоит из 3 частей и будет влит в develop после завершения.
https://gitlab.com/terrakok/gitlab-client/merge_requests/115
Поменяйте в вашем mr, что это относится к model.
привет ещё раз, насчёт mr. Я для вас создал базовый mr, который состоит из 3 частей и будет влит в develop после завершения.
https://gitlab.com/terrakok/gitlab-client/merge_requests/115
Поменяйте в вашем mr, что это относится к model.
ES
такая структура используется для простоты и удобства review