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/#218] 2nd my qa #219

Merged
merged 7 commits into from
Aug 24, 2024
Merged

[Fix/#218] 2nd my qa #219

merged 7 commits into from
Aug 24, 2024

Conversation

hyeeum
Copy link
Contributor

@hyeeum hyeeum commented Aug 23, 2024

⛳️ Work Description

  • 폴더 store 이미지 업데이트
  • 폴더 컴포넌트 여백 기기대응
  • 식당 구경하러 가기 body3 폰트 변경
  • 버튼 하나 모달의 버튼 medium으로 변경

📸 Screenshot

이전과 동일합니다!

📢 To Reviewers

  • 2차 QA 화이팅 :)

@hyeeum hyeeum added FIX 🛠️ 버그 및 오류 해결 UI 📱 UI 작업 혜음 🥖 labels Aug 23, 2024
@hyeeum hyeeum added this to the 2차 스프린트 QA milestone Aug 23, 2024
@hyeeum hyeeum requested review from chattymin and 0se0 August 23, 2024 17:12
@hyeeum hyeeum self-assigned this Aug 23, 2024
Copy link
Member

@chattymin chattymin left a comment

Choose a reason for hiding this comment

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

굿~ 고생했어요!

@@ -195,7 +195,7 @@ fun MyJogboDetailScreen(
title = jogboTitle,
chips = jogboChips,
userNickname = userNickname,
shareJogbo = updateShareDialogState
shareJogboDialog = updateShareDialogState
Copy link
Member

Choose a reason for hiding this comment

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

State는 네이밍 맞추는게 어떨까용~

Suggested change
shareJogboDialog = updateShareDialogState
shareJogboDialogState = updateShareDialogState

Copy link
Contributor Author

@hyeeum hyeeum Aug 23, 2024

Choose a reason for hiding this comment

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

아 이거 전 PR에도 비슷하게 코멘트 달렸는데...또..놓쳤네요..제가....

@chattymin chattymin merged commit 7b6f599 into develop Aug 24, 2024
@hyeeum hyeeum deleted the fix/#218-2nd-my-qa branch August 24, 2024 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FIX 🛠️ 버그 및 오류 해결 UI 📱 UI 작업 혜음 🥖
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fix] 2nd 디자인 QA
2 participants