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

Feat/app demo #2

Merged
merged 196 commits into from
Aug 7, 2024
Merged

Feat/app demo #2

merged 196 commits into from
Aug 7, 2024

Conversation

xeewii
Copy link
Contributor

@xeewii xeewii commented Jul 3, 2024

https://github.com/rees46/planning/issues/823

Реализовано демо приложение, показывающее возможности текущего сдк:
скелет приложения, блок рекомендаций, карточка товаров, экран home, экран корзины

@xeewii xeewii requested a review from TorinAsakura July 3, 2024 16:27
@xeewii xeewii self-assigned this Jul 3, 2024
@TorinAsakura
Copy link
Member

Следующий этап это формирование изолированного UI слоя по канонам. Чтобы не расписывать всё то, что я уже расписывал, вот ссылка на аналогичное ревью в iOS демо репе rees46/demo-ios#1 (review)

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

В общем в ui я пока вынесла recyclerView, buttons и countCard (это элемент, где счетчик, который можно менять по + и -). В recyclerView создала базовые элементы списка, в котором вся логика. И для нужных мне списков наследовалась от них. Например, адаптер стал будто фабрикой, в которой я просто создаю нужный элемент, а подключение, биндинг и тд уже происходит одинаково в базовом адаптере. Так же эти отдельные списки вынесла тоже в ui (хотя может стоило оставить в фичах, где они и были). Кнопки также одинаковые для всего проекта, настраивать их там тоже можно.

Это если короткое описание, так я эти изменения залила и заиспользовала. Правильно ли делаю, или как нужно всё-таки? На примеры опиралась, но может не так адаптировала их(

Я тебе дал исчерпывающий пример к имплементации. Сейчас из всего что ты сделала правильным является только вынос кнопок, да и то пока я не видел как ты их вынесла

@xeewii xeewii requested a review from TorinAsakura August 7, 2024 19:30
@TorinAsakura TorinAsakura merged commit d53469c into master Aug 7, 2024
@TorinAsakura TorinAsakura deleted the feat/app-demo branch August 7, 2024 21:24
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.

3 participants