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/#113] 필터링 재설정 서버통신 구현 #132

Merged
merged 13 commits into from
Jul 18, 2024

Conversation

Hyobeen-Park
Copy link
Member

⛳️ Work Description

  • 필터링 정보 API 연결
  • 필터링 정보 수정 API 연

📸 Screenshot

Screen_Recording_20240718_201516_Terning-Android.mp4

📢 To Reviewers

  • 너무 오래걸렸네요😂😂

@Hyobeen-Park Hyobeen-Park added FEAT ✨ 새로운 기능 구현 효빈💚 효빈 labels Jul 18, 2024
@Hyobeen-Park Hyobeen-Park self-assigned this Jul 18, 2024
Copy link
Member

@leeeyubin leeeyubin left a comment

Choose a reason for hiding this comment

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

홈 뷰가 많이 복잡했을 텐데 야무지게 다 해낸 효빈언니 칭찬합니다!

Comment on lines 17 to 24
) {
fun toHomeFilteringInfoModel(): HomeFilteringInfoModel =
HomeFilteringInfoModel(
grade = this.grade,
workingPeriod = this.workingPeriod,
startYear = startYear,
startMonth = startMonth,
)
Copy link
Member

Choose a reason for hiding this comment

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

startYear랑 startMonth는 this를 빼준 의도가 있나용?

Copy link
Member Author

Choose a reason for hiding this comment

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

헉스 통일시키겠습니다!!

Comment on lines 184 to 185
isEnabled = currentGrade in 0..3 && currentWorkingPeriod in 0..2
)
Copy link
Member

Choose a reason for hiding this comment

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

요거 상수화로 빼주면 더 좋을 것 같아요!

Copy link
Member Author

Choose a reason for hiding this comment

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

오 좋아요!!

Comment on lines 7 to +9
val showSearchResults: Boolean = false,
val existSearchResults: Boolean = false,
val searchResults: List<InternData> = emptyList(),
// val searchResults: List<InternData> = emptyList(),
Copy link
Member

Choose a reason for hiding this comment

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

주석 지워주세용

Copy link
Member Author

Choose a reason for hiding this comment

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

앗 이거는 제 코드가 아니었어가지고,,, 머지 다 되면 없어질 것 같아요..!!

Copy link
Member

@boiledEgg-s boiledEgg-s left a comment

Choose a reason for hiding this comment

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

LGTM💣💣💣

@Hyobeen-Park Hyobeen-Park merged commit defac4e into develop Jul 18, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FEAT ✨ 새로운 기능 구현 효빈💚 효빈
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] 필터링 재설정 뷰 / 필터링 서버 통신 구현
3 participants