Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/ Demo store #1

Merged
merged 86 commits into from
Aug 7, 2024
Merged

Feature/ Demo store #1

merged 86 commits into from
Aug 7, 2024

Conversation

DenielGrin
Copy link
Contributor

@DenielGrin DenielGrin changed the title Feature/Demo store Feature/ Demo store Jul 3, 2024
@DenielGrin DenielGrin self-assigned this Jul 10, 2024
@DenielGrin
Copy link
Contributor Author

@TorinAsakura Все замечания исправил, лишнее выкинул, новое привнёс. Избавился от лишних синглтонов, всё что было можно и нужно перевёл на DI.
Так же все коды для инициализации сдк, загрузки блоков товаров и прочее, всё утащил в переменные окружение и под них вывел отдельный файл в кор слое.
Разобрался с резолвером, удобная штука, напоминает Koin.
Возможно, что то упустил в расположении файлов, жду критики

@DenielGrin DenielGrin requested a review from TorinAsakura July 24, 2024 16:23
Copy link
Member

@TorinAsakura TorinAsakura left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Рассмотри использование Lazy*Stack
  2. Если со структурой мы более или менее разобрались то UI как был разбросанной кучей так и остался - вот пример того, как он должен формироваться https://github.com/torin-asakura/dream-team/tree/master/ui - зависимости этих пакетов я уж думаю сам найдёшь по коду
  3. Далее, у тебя во вьюхах экранов опять происходит стройка, хотя я тебе уже который раз объяснял, что во вьюхах ты должен просто импортировать куски от этих экранов и производить только их сборку, а вся стройка должна происходит внутри этих кусков - вот пример как у нас выглядят страницы (структурно) https://github.com/torin-asakura/dream-team/tree/master/landing/pages - такой же смысл должен быть и тут
  4. Если посмотришь на уровень выше - увидишь фрагменты - это из области микрофронтов, кроме самих микрофронтов, но, суть примерно та же - каждый бизнес кусок поделён на фрагменты, вся стройка этих кусков происходит внутри фрагментов из юай компонентов которые живут также отдельным слоем, а в самих страничках происходит только сборка из фрагментов, всё

@TorinAsakura
Copy link
Member

@DanielGreenEngineer также хочу обратить твоё внимание, что енамы ты именуешь через конвенциональный нейминг, а протоколы и стракты нет, стракты вообще не имменуешь

@DenielGrin
Copy link
Contributor Author

@DanielGreenEngineer также хочу обратить твоё внимание, что енамы ты именуешь через конвенциональный нейминг, а протоколы и стракты нет, стракты вообще не имменуешь

Добавил суффиксы всем страктам, а протоколы оставил без суффиксов, у них в имени класса указал "Протокол"

@DenielGrin DenielGrin requested a review from TorinAsakura August 5, 2024 13:05
@DenielGrin
Copy link
Contributor Author

@TorinAsakura Все замечания исправил, вьюхи разбил на мелкие кусочки, так же раскидал их по пакетам в соответствующих фичах. Цвета и размеры утащил в константы.

Copy link
Member

@TorinAsakura TorinAsakura left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Продолжаем марлезонский баллет…

@DenielGrin DenielGrin requested a review from TorinAsakura August 7, 2024 15:54
@TorinAsakura TorinAsakura merged commit 25ec303 into master Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants