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] 콕 찌르기 개편 #372

Merged
merged 10 commits into from
Jun 6, 2024

Conversation

lsj8706
Copy link
Member

@lsj8706 lsj8706 commented Jun 2, 2024

🌴 PR 요약

  • 콕 찌르기 서비스를 개선했어요.

🌱 PR Point

  1. 콕 찌르기 메인 화면에서 친구 추천 섹션을 2개 -> 3개로 변경 (같인 기수, mbti, 주량)
  2. 콕 찌르기에서 사용하고 있는 UI Component에 익명인 경우를 대응했어요 (실명 숨기기 등)
  3. 익명인 사람을 찔렀을 때 PokeNum이 5 or 11이 되는 경우 로띠 뷰를 보여주도록 기능을 추가했어요.
    -> 메인뷰, 내 친구 뷰, 내 친구 리스트 뷰, 알림 뷰에 대응했어요. (더 필요한 곳이 있나..? 기억이 안 나요)

📌 참고 사항

  • 익명일 시 profileImage를 변경하는 것은 아직 안했어요. (서버에서 API 필드 이름 확정해주면 대응 가능)
  • 빅 PR이네요. 빠른 작업을 위해 코드 퀄리티는 먼 곳으로 가버렸습니다.

📸 스크린샷

익명친구가 5콕(단짝친구)이 되었을 때 로띠

Simulator.Screen.Recording.-.iPhone.15.Pro.-.2024-06-02.at.17.59.24.mp4

익명친구가 11콕(천생연분)이 되었을 때 로띠

Simulator.Screen.Recording.-.iPhone.15.Pro.-.2024-06-02.at.18.23.54.mp4

@lsj8706 lsj8706 added the Feat 새로운 기능 구현 label Jun 2, 2024
@lsj8706 lsj8706 requested review from elesahich and ffalswo2 June 2, 2024 09:25
@lsj8706 lsj8706 self-assigned this Jun 2, 2024
Copy link

height bot commented Jun 2, 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.

Copy link
Contributor

@elesahich elesahich left a comment

Choose a reason for hiding this comment

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

가즈아 가즈아

@lsj8706 lsj8706 merged commit cab4e98 into sopt-makers:develop Jun 6, 2024
@lsj8706 lsj8706 deleted the feat/콕-찌르기-개편 branch June 6, 2024 01:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feat 새로운 기능 구현 size/XXL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants