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

Refactor [#242] 추천친구 리팩토링 #268

Open
wants to merge 16 commits into
base: develop
Choose a base branch
from

Conversation

chaentopia
Copy link
Collaborator

⛏ 작업 내용

  • 추천친구와 검색 부분 리팩토링 진행했습니다.
  • 전반적인 구조를 바꾸는 건 어려울 것 같아서.. 우선은 불필요한 코드 삭제와 간단한 수정을 진행했습니다.

📌 PR Point!

  • x

📸 스크린샷

구현 내용 스크린샷
화면종류 x

✅ Issue

Resolved #242

Copy link
Member

@Zoe0929 Zoe0929 left a comment

Choose a reason for hiding this comment

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

P5
전체적으로 중복되는 코드들을 foreach 사용해서 간결하게 작성한 게 좋네요!!
고생하셨습니다. 구조를 바꾸는 건 다시 또 얘기해서 적용해보는 게 좋을 것 같아요 💖💖 파이팅 🔥

Copy link
Contributor

@Heyjooo Heyjooo left a comment

Choose a reason for hiding this comment

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

P5
수고하셨습니다!~!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

[Refactor] 추천친구 및 검색 리팩토링
3 participants