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: 커피챗 QA #1665

Merged
merged 15 commits into from
Nov 6, 2024
Merged

feat: 커피챗 QA #1665

merged 15 commits into from
Nov 6, 2024

Conversation

seojisoosoo
Copy link
Member

@seojisoosoo seojisoosoo commented Nov 5, 2024

🤫 쉿, 나한테만 말해줘요. 이슈넘버

🧐 어떤 것을 변경했어요~?

  • qa를 진행했어요.
    • [오픈하기] 진행방식 default 값이 잘못 입력되어 있음
    • [오픈하기] 타이틀이 헤더와 겹침
    • [수정하기] 모든 수정 내용이 반영되지 않음
    • [본인 상세] 삭제 클릭시, dialog의 삭제하기 btn 레이아웃 깨짐
    • [본인 상세] 메뉴 클릭시, dim처리 안됨
  • 예외처리도 해주었어요.
    • url을 변경해서 upload페이지에 접근한 유저가 커피챗을 이미 올린 유저라면 모달을 띄워주었어요.
  • 서버와의 소통을 통해 변경된 타입이 있는 경우 수정해주었어요.

🤔 그렇다면, 어떻게 구현했어요~?

  • 쿼리키 무효화하지 않은 부분 없는지 찾아서 수정했어요.
  • 데이터 받아오는 타입이 틀렸거나 디폴트값 설정이 잘못된 경우를 수정했어요.
  • mds 버전 업

❤️‍🔥 당신이 생각하는 PR포인트, 내겐 매력포인트.

  • 수정, 삭제 버튼이 담긴 드롭다운 or 바텀시트 -> 삭제하시겠습니까?다이얼로그 이렇게 두 모달이 겹치면서, 오버레이가 사라지지 않는 이슈가 있었어요. (보기엔 사라져보이지만, 뒤의 컴포넌트들이 클릭되지 않음.)
스크린샷 2024-11-05 오후 10 37 09
  • 기존에도 있었던 이슈였는데요, 우선은 드롭다운과 바텀시트에 modal={false}를 props로 내려주고, Dialog.Overlay 사용없이, Overlay 컴포넌트를 만들어서 Dialog.Content의 백그라운드로 보일 수 있게 넣었습니다.

📸 스크린샷, 없으면 이것 참,, 섭섭한데요?

Copy link

height bot commented Nov 5, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

@github-actions github-actions bot requested a review from pepperdad November 5, 2024 13:39
@seojisoosoo seojisoosoo self-assigned this Nov 5, 2024
Copy link

github-actions bot commented Nov 5, 2024

✨✨ 스토리북으로 확인하기 ✨✨

Copy link

github-actions bot commented Nov 5, 2024

🚀 프리뷰 배포 확인하기 🚀

https://cd517d07.sopt-internal-dev.pages.dev

@pull-request-size pull-request-size bot added size/L and removed size/M labels Nov 5, 2024
Copy link
Member

@pepperdad pepperdad left a comment

Choose a reason for hiding this comment

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

굿입니다!!!!!!

@seojisoosoo seojisoosoo added this pull request to the merge queue Nov 6, 2024
Merged via the queue into main with commit 8a020c5 Nov 6, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: 커피챗 qa
2 participants