@terrakok
если есть вопросы по комментариям, то их сюда задавать или в MR?

Issue и MergeRequest во некоторых местах апи добавляются как target, а в некоторых местах как target_id + target_type, и получается, что Target существует независимо от Todo, а Todo просто содержит ссылку.

Например из Events ["target_id":160, "target_type":"Issue"], видно, что Issue/MergeRequest это реализация target.
Поэтому я считаю, что создавать IssueTarget и MergeRequestTarget это дублирование Issue и MergeRequest.

логично, что туда, куда комментарий написали)

понял, тогда Target, Issue и MergeRequest надо перенести в пакет entity.target

хорошо.

лучше вопросы здесь задавать или в mr?

Может ещё это обсудим сразу?

получилось так, что и здесь спросил, и в mr 🙁

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

Привет. Как проавильно реализовать слушателя для DialogFragment? Задача с navigationdrawer. Как я понял в  BaseFagment метод showConfirmDialog(title: String? = null, msg: String, confirm: () -> Unit) должен вызывать ConfirmDialog в котором создается AlertDialog.

Да, надо создать наследника фрагмент диалога. А в онаттач кастовать:
Паррент фрагмент к интерфейсу слушателя, если это не так, то каствать активити

А интерфес ConfirmDialog.OnClickListener должен реализовывать BaseFragment или NavigationDrawerFragment?

Base

И лучше в интерфейсе не метод сделать, а лямбду, наверное

Надо посмотреть)

protected fun showConfirmDialog(title?, msg?, confirm {})  Не понятно как вызвать confirm()

Надо ещё передать тег, и сохранить лямбду

При создании диалога передать лямбду в бандл

Передавать лямбду в бандл это какая-то джедайская техника прям)

А, ещё я понял, что не стоит все это в базовом классе делать)