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

친구 추가 삭제 시, 전역 정보 반영 및 에러 수정 #132

Merged
merged 7 commits into from
Dec 4, 2022

Conversation

ktmihs
Copy link
Member

@ktmihs ktmihs commented Dec 4, 2022

📕 제목

📗 작업 내용

구현 내용 및 작업 했던 내역

  • 함수 호출을 안 해주어 실행 안 됐던 문제 해결
  • 함수로 잘못 불렀던 로직 수정
  • friendlist 내부 id 추가
  • 변수로 잘못 사용된 uuid를 함수 호출로 변경
  • 친구 검색 자동 완성 api 연결(전역 값도 변경해줌)
  • 채팅방 클릭 시, 내부 어떤 요소를 클릭하든 id를 찾을 수 있도록 변경

📘 PR 특이 사항

PR을 볼 때 주의깊게 봐야하거나 말하고 싶은 점

- 함수 호출을 안 해주어 실행 안 됐던 문제 해결
- 함수로 잘못 불렀던 로직 수정
- friendlist 내부 id 추가
- 변수로 잘못 사용된 uuid를 함수 호출로 변경
- 내부 어떤 요소를 클릭하든 id를 찾을 수 있도록 변경
@ktmihs ktmihs added 프론트엔드 프론트엔드 버그 리팩토링 Refactoring 기능 기능 추가 labels Dec 4, 2022
@ktmihs ktmihs added this to the [ 3주차 ] 22.11.28 ~ 22.12.04 milestone Dec 4, 2022
@ktmihs ktmihs self-assigned this Dec 4, 2022
Comment on lines +28 to +30
friendList.map((friend: friendType) => (
<FriendItem friend={friend} key={friend.id} />
))
Copy link
Collaborator

Choose a reason for hiding this comment

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

타입 활용 너무 좋아요~~ 배워갑니다

@JJongBin JJongBin merged commit a50397e into boostcampwm-2022:develop Dec 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
기능 기능 추가 리팩토링 Refactoring 버그 프론트엔드 프론트엔드
Projects
Status: 🎄 Done
Development

Successfully merging this pull request may close these issues.

2 participants