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/add get client shopping cart api method #86

Merged
merged 5 commits into from
Dec 19, 2024

Conversation

looee1q
Copy link
Contributor

@looee1q looee1q commented Dec 16, 2024

@looee1q looee1q self-assigned this Dec 16, 2024
@looee1q
Copy link
Contributor Author

looee1q commented Dec 17, 2024

Issue_719

@looee1q looee1q force-pushed the feat/add-get-client-shopping-cart-api-method branch from b035511 to 52c8085 Compare December 17, 2024 16:29
@looee1q looee1q force-pushed the feat/add-get-client-shopping-cart-api-method branch from 52c8085 to 28a0f29 Compare December 17, 2024 20:20
@looee1q looee1q requested a review from TorinAsakura December 17, 2024 20:52
…CallbackListener in CartManager to using its own callbacks.
@looee1q
Copy link
Contributor Author

looee1q commented Dec 18, 2024

@TorinAsakura Исправил часть комментариев.

По части комментариев относительно использования корутин и отказа от колбеков следующие соображения:
Сейчас в проекте все взаимодейтсвие с сетью обеспечивается через SendNetworkMethodUseCase и NetworkRepositoryImpl с использованием HttpURLConnection, который являлся стандартом для работы с сетью в андройд когда-то очень давно. Сейчас же для целей взаимодействия с сетью используются либо Retrofit либо Ktor, которые из коробки поддерживают отличную совместимость с корутинами.

Переход на использование корутин и отказ от кобеков считаю правильным вектором, которому следует придерживаться. Однако, чтобы не сломать текущий код, этот процесс следует внедрять постепенно и параллельно существующему способу работы с сетью, и осуществлять его в рамках отдельных issue.

Обеспечение сетевого взаимодействия в андройде сейчас используется в основном либо через Retrofit, либо Ktor, являющиеся стандартом для таких задач. Поэтому так же предлагаю рассмотреть возможность внедрения одной из этих библиотек, которые обеспечивают отличную совместимость с корутинами и берут асинхронную работу с сетью на себя.

@looee1q
Copy link
Contributor Author

looee1q commented Dec 18, 2024

Подверждение правильности работы метода после нового коммита:

Issue_719_new

@looee1q looee1q requested a review from TorinAsakura December 18, 2024 11:15
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.

жду последних правок

@looee1q looee1q requested a review from TorinAsakura December 19, 2024 09:52
@TorinAsakura TorinAsakura merged commit 990c4bc into master Dec 19, 2024
1 check passed
@TorinAsakura TorinAsakura deleted the feat/add-get-client-shopping-cart-api-method branch December 19, 2024 11:08
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