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

πŸ”€ :: (#214) ν”„λ‘œμ νŠΈ μ•„ν‚€ν…μ²˜ μˆ˜μ • #219

Merged
merged 123 commits into from
Jun 13, 2024

Conversation

Chaejongin12
Copy link
Collaborator

πŸ’‘ κ°œμš”

κΈ°μ‘΄ μ•„ν‚€ν…μ²˜μ˜ 단점을 κ°œμ„ ν•˜κΈ° μœ„ν•΄ ꡬ글 ꢌμž₯ μ•„ν‚€ν…μ²˜λ‘œ ν”„λ‘œμ νŠΈμ˜ 섀계λ₯Ό μˆ˜μ •ν•΄μ•Όν•©λ‹ˆλ‹€.

πŸ“ƒ μž‘μ—…λ‚΄μš©

ꡬ글 ꢌμž₯ μ•„ν‚€ν…μ²˜μ˜ κ·œμΉ™μ„ 지킀기 μœ„ν•΄ μ΅œλŒ€ν•œ μ•„ν‚€ν…μ²˜λ₯Ό λ³€κ²½ν•΄λ΄€μŠ΅λ‹ˆλ‹€.

πŸ”€ 변경사항

data λͺ¨λ“ˆμ— Entity둜 λ§€ν•‘ν•˜λŠ” Mapper 생성
model λͺ¨λ“ˆμ— Entity 생성
feature convention μ˜μ‘΄μ„± μˆ˜μ •
Request, Response network λͺ¨λ“ˆλ‘œ 이동
DataSource μ•„ν‚€ν…μ²˜ 변경사항 적용
Repository 맀핑 둜잭 μΆ”κ°€

πŸ™‹β€β™‚οΈ μ§ˆλ¬Έμ‚¬ν•­

μ œκ°€ 잘λͺ» κ΅¬ν˜„ν•œ 뢀뢄이 μžˆκ±°λ‚˜ λΆ€μžμ—°μŠ€λŸ¬μš΄ 넀이밍, λΆˆν•„μš”ν•œ 곡백이 μžˆλ‹€λ©΄ λ¦¬λ·°ν•΄μ£Όμ„Έμš”.

:: GetUserListUseCase invoke function runCatching kotlin μ ‘κ·Όμž μ‚­μ œ
Copy link
Member

@audgns10 audgns10 left a comment

Choose a reason for hiding this comment

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

μˆ˜κ³ ν•˜μ…¨μŠ΅λ‹ˆλ‹€

:: Event, errorHandling 파일 주석 μ‚­μ œ
@Chaejongin12 Chaejongin12 merged commit 1f3e9ca into develop Jun 13, 2024
1 check passed
@Chaejongin12 Chaejongin12 deleted the refactor/#214_change_project_architecture branch June 13, 2024 00:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0️⃣ Priority: Critical μš°μ„ μˆœμœ„ - κΈ΄κΈ‰!!!!!
Projects
None yet
4 participants