Сделал тестовое задание, видимо работодателям реализация не понравилась, хочу спросить

у вас про выбранный мною подход.

По заданию нужно распарсить один json с сервера и представить его в приложении.
Json - это список сотрудников, у каждого из которых есть личные данные(имя/возраст/...) и профессия(либо список профессий).

Представить нужно было как угодно, лишь бы юзер мог получить информацию:

- Список сотрудников
- Список сотрудников, принадлежащих к конкретной профессии
- Информацию о конкретном сотруднике.

Я сделал через Single Activity + NavComponent + SharedVM. Три фрагмента, каждый подписан на свои MutableLiveData из SharedVM и каждый отвечает за свою часть задания. Логика работы с данными следующая:

-Стартуем приложение
-В контейнер Activity(FrameLayout) NavComponent сеттит первый фрагмент(Start destination). Фрагмент подписывается на свои MutableLiveData, но данных пока нет.
-В Activity инжектится презентер.
-В конструкторе презентера инжектятся Employee и Speciality репозитории.
-Презентер коллбеком просит Activity показать лоадер и просит Employee репозиторий подгрузить json.
-Репозиторий стучится в Api(инжектировано), ассинхронно получает ответ, возвращает его в презентер.
-Презентер коллбеком возвращает ответ(модель всего JSON) в Activity.

И вот скорее всего дальше я начал делать дичь.

- Activity обращается к презентеру с просьбой закешировать в репозиториях список сотрудников и список специальностей, передавая ему тот же ответ, который он вернул в предыдущем пункте.
- Презентер обрабатывает Json и предоставляет обработанные результаты репозиториям.
- Репозитории кешируют в себе данные(репозитории - синглтоны).
- Activity создаёт SharedViewModel.
- Внутри SharedViewModel инжектятся Employee и Speciality репозитории.
- Внутри SharedViewModel создаются несколько MutableLiveData, которые содержат списки сотрудников и специальностей.
- В эти LiveData сетятся данные из репозиториев.
- Фрагменты, которые при создании подписаны на свои MutableLiveData, начинают получать данные.

Вопрос в целом в том, является ли такой подход хотя бы приемлимым, и что можно улучшить?

Мне кажется тут презентер нафиг не нужен вообще, но убрать его пока не могу. Могу переложить его обязаности на SharedVM, но тогда я Single Responsibility вообще проверчу(и без того нарушено), т.к VM у меня и без того слишком дохрена всего делает(отвечает за контент трех фрагментов разом).

GitHub, мб кому интересно будет:

https://github.com/KirstenLy/TestTask

знаю для кого это задание, тоже его делал)))