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] network 기초 세팅, RetrofitModule, ServiceModule 구현 #25

Merged
merged 35 commits into from
Jan 31, 2024

Conversation

murjune
Copy link
Member

@murjune murjune commented Jan 29, 2024

관련 이슈

  • closed #이슈넘버
  • discord로 실시간 리뷰함~

컨벤션 정한거

dto

  • xxxResponse, xxxRequest
  • feature에 관련된 package 명에 관련 Dto 모아두기
  • 중복되는 Dto는 가장 최상단에 놓기

json Parser

  • kotlinx-serialization 로 결정!

@murjune murjune requested a review from ham2174 January 29, 2024 17:57
@murjune murjune self-assigned this Jan 29, 2024
@murjune murjune force-pushed the feature/set-network-module branch 2 times, most recently from b8817c1 to d1b8708 Compare January 29, 2024 18:04
@murjune murjune force-pushed the feature/set-network-module branch from d1b8708 to c356314 Compare January 29, 2024 18:05
@murjune murjune force-pushed the feature/set-network-module branch from 55c8947 to 5680ffe Compare January 29, 2024 18:22
@murjune murjune changed the title Feature/set network module [Feat] network 기초 세팅, RetrofitModule, ServiceModule 구현 Jan 29, 2024
Copy link
Member Author

@murjune murjune left a comment

Choose a reason for hiding this comment

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

Trouble Shooting

Comment on lines +18 to +24
getByName("release") {
buildConfigField(
"String",
"FUNCH_DEBUG_BASE_URL",
properties.getProperty("FUNCH_DEBUG_BASE_URL"),
)
}
Copy link
Member Author

Choose a reason for hiding this comment

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

release 버전도 buildConfig 설정 해줘야함

Copy link
Collaborator

Choose a reason for hiding this comment

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

CI 오류 해결을 위해 설정 👍

@murjune murjune requested review from a team and removed request for a team and ham2174 January 29, 2024 19:46
Copy link
Collaborator

@ham2174 ham2174 left a comment

Choose a reason for hiding this comment

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

LGTM👍👍

Comment on lines +18 to +24
getByName("release") {
buildConfigField(
"String",
"FUNCH_DEBUG_BASE_URL",
properties.getProperty("FUNCH_DEBUG_BASE_URL"),
)
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

CI 오류 해결을 위해 설정 👍

@murjune murjune force-pushed the feature/set-network-module branch from b5d0380 to c6b17da Compare January 31, 2024 20:26
@murjune murjune merged commit 62bb831 into develop Jan 31, 2024
1 check passed
@murjune murjune deleted the feature/set-network-module branch January 31, 2024 20:34
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