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

[FIX/#236] 홈 뷰 / 1차 QA 반영 #243

Merged
merged 15 commits into from
Sep 13, 2024
Merged

[FIX/#236] 홈 뷰 / 1차 QA 반영 #243

merged 15 commits into from
Sep 13, 2024

Conversation

Hyobeen-Park
Copy link
Member

⛳️ Work Description

  • 뷰 크기 점검 및 수정
  • 곧 마감 공고 - 기업 명 추가
  • 다이얼로그 코드 수정
  • 라이팅 수정

📸 Screenshot

Screen_Recording_20240912_233829_terning.mp4

📢 To Reviewers

  • 1차 QA와 지금까지 언급됐던 추가 수정사항들 반영했습니다!
  • 아직 QA를 다 반영하지는 못했는데 서버통신 수정된 부분은 빠르게 PR 날리는게 좋을 것 같아서 (+남은 QA가 고민할 시간이 필요해서...ㅎㅎㅎ) 일단 먼저 날립니다!! 남은 수정사항은 새 브랜치에서 할까 생각중인데 혹시 이 브랜치에서 추가적으로 수정사항 발생하면 노티드릴게요..!!

@Hyobeen-Park Hyobeen-Park added FIX 🔨 버그 및 오류 해결 효빈💚 효빈 labels Sep 12, 2024
@Hyobeen-Park Hyobeen-Park added this to the 2차 스프린트 작업 milestone Sep 12, 2024
@Hyobeen-Park Hyobeen-Park self-assigned this Sep 12, 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.

확인했습니당!! 1차 QA반영 빠르군여 ㄷㄷ 최고!!!

Comment on lines 87 to 90
Text(
text = dateDeadline,
style = TerningTheme.typography.detail0,
color = TerningMain,
color = if (dateDeadline == "지원마감") Grey300 else TerningMain,
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 88 to 91
LaunchedEffect(key1 = true) {
viewModel.getProfile()
viewModel.getFilteringInfo()
}
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.

점점 코리 달아줄게 없어지네요ㅎㅎ 나머지도 화이팅해서 해치웁시닷!!

@@ -86,7 +87,7 @@ fun InternItem(
Text(
text = dateDeadline,
style = TerningTheme.typography.detail0,
color = TerningMain,
color = if (dateDeadline == stringResource(id = R.string.intern_apply_closed)) Grey300 else TerningMain,
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
Contributor

@arinming arinming 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 +182 to +186
if (homeState.homeRecommendDialogVisibility) {
with(homeState.homeInternModel) {
if (this != null) {
if (isScrapped) {
ScrapCancelDialog(
Copy link
Contributor

Choose a reason for hiding this comment

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

깔꼼

@Hyobeen-Park Hyobeen-Park merged commit 5f590c8 into develop Sep 13, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FIX 🔨 버그 및 오류 해결 효빈💚 효빈
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] 홈 뷰 / QA 및 추가 수정사항 반영
4 participants