-
Notifications
You must be signed in to change notification settings - Fork 1
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/#113] 필터링 재설정 서버통신 구현 #132
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
홈 뷰가 많이 복잡했을 텐데 야무지게 다 해낸 효빈언니 칭찬합니다!
) { | ||
fun toHomeFilteringInfoModel(): HomeFilteringInfoModel = | ||
HomeFilteringInfoModel( | ||
grade = this.grade, | ||
workingPeriod = this.workingPeriod, | ||
startYear = startYear, | ||
startMonth = startMonth, | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
startYear랑 startMonth는 this를 빼준 의도가 있나용?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
헉스 통일시키겠습니다!!
isEnabled = currentGrade in 0..3 && currentWorkingPeriod in 0..2 | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
요거 상수화로 빼주면 더 좋을 것 같아요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오 좋아요!!
val showSearchResults: Boolean = false, | ||
val existSearchResults: Boolean = false, | ||
val searchResults: List<InternData> = emptyList(), | ||
// val searchResults: List<InternData> = emptyList(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
주석 지워주세용
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
앗 이거는 제 코드가 아니었어가지고,,, 머지 다 되면 없어질 것 같아요..!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM💣💣💣
⛳️ Work Description
📸 Screenshot
Screen_Recording_20240718_201516_Terning-Android.mp4
📢 To Reviewers