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/#239] 공고 상세 뷰 / QA 수정사항 반영 #249

Merged
merged 10 commits into from
Sep 16, 2024

Conversation

arinming
Copy link
Contributor

⛳️ Work Description

  • 화면 background 추가
  • padding 값 조정
  • 서버에 맞게 데이터 수정
  • 스크랩 다이얼로그 로직 수정

📸 Screenshot

239.mp4

📢 To Reviewers

  • 지원 사이트로 이동하기 이슈만 해결하면 되는데 이 이슈가 제 에뮬에서는 없고 정상작동해서,.,.,., 흡 실기기 문제일까욤

@arinming arinming added this to the 2차 스프린트 작업 milestone Sep 14, 2024
@arinming arinming self-assigned this Sep 14, 2024
@arinming arinming linked an issue Sep 14, 2024 that may be closed by this pull request
3 tasks
@arinming arinming changed the title Fix/#239 announcement [Fix/#239] 공고 상세 뷰 / QA 수정사항 반영 Sep 14, 2024
@arinming arinming changed the title [Fix/#239] 공고 상세 뷰 / QA 수정사항 반영 [FIX/#239] 공고 상세 뷰 / QA 수정사항 반영 Sep 14, 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.

큐에이 반영 완전 최고에요!!! 수고했습니다:)
사이트 이동하기 관련 이슈 해결하면 알려주세요!!

Copy link
Member

Choose a reason for hiding this comment

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

제가 아린언니 브렌치 들어가서 돌려봤는데 실기기 문제인 것 같아요..! 공기계로 돌려보니 안 뜨더라구요ㅜㅜ

130번째 줄에 아래와 같이 수정하면 웹 뷰 잘 뜨는 것 같습니당..!
확인부탁드려요!

val context = LocalContext.current

if (internUiState.showWeb) {
    CustomTabsIntent.Builder().build().launchUrl(context,internInfo.url.toUri())
}

Copy link
Member

@leeeyubin leeeyubin Sep 14, 2024

Choose a reason for hiding this comment

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

추가로.. 제가 말한대로 작성 시 백버튼 눌렀을 때 다시 웹 뷰가 뜨는 이슈가 있는 것 같은데,, 저도 정확히 보지 않아서 뭐가 문제인지 확실하진 않지만,, 이 부분은 MyPageRoute 한번 참고해보면 좋을 것 같아요..!

Copy link
Contributor Author

@arinming arinming Sep 15, 2024

Choose a reason for hiding this comment

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

새삼스럽지만 사랑합니다 ㅜㅜ
수정했는데, 실기기에서 잘 동작하는지 확인해주심 감사 🥹

Copy link
Member

Choose a reason for hiding this comment

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

확인했습니당 잘 동작하네요!! 수고했어요!!🙂 @arinming

Comment on lines 144 to +149
Column(
modifier = modifier.fillMaxHeight(),
modifier = modifier
.statusBarsPadding()
.navigationBarsPadding()
.fillMaxHeight()
.background(White),
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

@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.

너무 고생하셨습니다~ 오류난 부분도 후딱 고쳐봅시다!

Copy link
Member

@Hyobeen-Park Hyobeen-Park left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~~!!! 오류 수정도 파이팅!!!!!!!

@arinming arinming requested a review from leeeyubin September 15, 2024 18:10
@arinming arinming added FIX 🔨 버그 및 오류 해결 아린💛 아린 labels Sep 15, 2024
@arinming arinming merged commit 5c6c360 into develop Sep 16, 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
Status: Done
Development

Successfully merging this pull request may close these issues.

[FIX] 공고 상세 뷰 / QA 수정사항 반영
4 participants