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] 멤버 coffee chat 관련 필드 반영 #540

Merged
merged 2 commits into from
Nov 9, 2024

Conversation

jun02160
Copy link
Member

@jun02160 jun02160 commented Nov 8, 2024

🐬 요약

멤버 프로필 쪽 coffee chat migration 시 미적용된 부분이 있어 마저 반영합니다.

👻 유형

PR의 유형에 맞게 체크해주세요!

  • 버그 수정
  • 기능 개발
  • 코드 스타일 수정 (formatting, local variables)
  • 리팩토링 (no functional changes, no api changes)
  • 빌드 관련 변경사항
  • CI 관련 변경사항
  • CD 관련 변경사항
  • 문서 내용 변경
  • Release
  • 기타... (다음 줄에 사유를 입력해주세요)

🍀 작업 내용

PR에 담긴 작업 내용을 작성해주세요!

  • 작업 1
  • 작업 2
  • 작업 3

🌟 관련 이슈

PR과 관련된 이슈 번호를 작성해주세요!

close: #00

@jun02160 jun02160 requested a review from dev-Crayon November 8, 2024 17:30
Copy link

height bot commented Nov 8, 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.

@jun02160 jun02160 self-assigned this Nov 8, 2024
@jun02160 jun02160 added ⛏ Fix 버그, 오류 해결 예준🐈 예준이 연 PR, Issue에 사용 labels Nov 8, 2024
Copy link
Contributor

@dev-Crayon dev-Crayon left a comment

Choose a reason for hiding this comment

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

LGTM! 빠르고 깔끔한 작업 감사합니다. 코드리뷰 하다가 생각이 든건데 멤버 프로필 조회 [타인, 본인]조회 사실 거의 똑같은 코드의 반복인 것 같네요 현재 커피챗 부분에서 조회는 하나의 API에서 타인과 본인에 따른 로직을 하나의 API 내부에서 처리하고 있는데 이쪽 방식으로 추후 리팩토링 해봐도 좋을 것 같습니다. 이번 스프린트가 조금 길고 수정사항도 많았는데 고생많으셨습니다.

차주 목요일에 플그 백엔드 회의에서 리프레쉬 기간에 리팩토링 진행할지 여부랑 리팩토링 우선순위 & 기능 개발 가이드라인 작성 등등 논의 나누어보면 좋을 것 같습니다.
감사합니다 :)

@jun02160
Copy link
Member Author

jun02160 commented Nov 9, 2024

@dev-Crayon 너무 좋은 의견 감사합니다!! 리프레시 기간 동안 리팩토링 달려봅시다 💪

@jun02160 jun02160 changed the title fix: 멤버 coffee chat 관련 필드 반영 [Fix] 멤버 coffee chat 관련 필드 반영 Nov 9, 2024
@jun02160 jun02160 merged commit 37e6839 into develop Nov 9, 2024
1 check passed
@jun02160 jun02160 deleted the fix/member_profile_coffeechat branch November 9, 2024 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⛏ Fix 버그, 오류 해결 size/S 예준🐈 예준이 연 PR, Issue에 사용
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants