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.

Sign up for GitHub

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

Jump to bottom

🔀 :: (#765) 플레이리스트 디테일 디자인 검수 #767

Merged
merged 12 commits into from
Jul 12, 2024
Merged

🔀 :: (#765) 플레이리스트 디테일 디자인 검수 #767

merged 12 commits into from
Jul 12, 2024

Conversation

yongbeomkwak
Copy link
Member

💡 배경 및 개요

lineHeight 누락된 것을 채워넣습니다.

Resolves: #765

📃 작업내용

  • playlistDetail lineHeight을 기입했습니다.
  • 이미지 선택 관련 로깅을 변경합니다.

🙋‍♂️ 리뷰노트

구현 시에 고민이었던 점들 혹은 특정 부분에 대한 의도가 있었다면 PR 리뷰의 이해를 돕기 위해 서술해주세요!

또한 리뷰어에게 특정 부분에 대한 집중 혹은 코멘트 혹은 질문을 요청하는 경우에 작성하면 좋아요!

e.g. 작업을 끝내야할 시간이 얼마 없어 확장성보다는 동작을 위주로 만들었어요! 감안하고 리뷰해주세요!

✅ PR 체크리스트

템플릿 체크리스트 말고도 추가적으로 필요한 체크리스트는 추가해주세요!

  • 이 작업으로 인해 변경이 필요한 문서가 변경되었나요? (e.g. XCConfig, 노션, README)
  • 이 작업을 하고나서 공유해야할 팀원들에게 공유되었나요? (e.g. "API 개발 완료됐어요", "XCConfig 값 추가되었어요")
  • 작업한 코드가 정상적으로 동작하나요?
  • Merge 대상 브랜치가 올바른가요?
  • PR과 관련 없는 작업이 있지는 않나요?

🎸 기타

@github-actions github-actions bot added the 0️⃣ Priority: Critical 우선순위 긴급 label Jul 10, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Jul 10, 2024 •
edited
Loading

✅ Successful finished SwiftLint

@github-actions GitHub Actions
Copy link

✅ Assign 자동 지정을 성공했어요!

@yongbeomkwak

@baekteun
Copy link
Member

PR 대상 잘못 지정하신거같아요

@yongbeomkwak
Copy link
Member Author

PR 대상 잘못 지정하신거같아요

PR 대상 맞는 것 같은데요?

@baekteun
Copy link
Member

#764
image

Base automatically changed from 762-fix-search-result-layout-mistake to develop July 11, 2024 14:37
@yongbeomkwak
Copy link
Member Author

#764 image
앗 번호 따라가다가 ..

@yongbeomkwak yongbeomkwak changed the base branch from develop to 632-inject-log-in-search July 11, 2024 14:41
Base automatically changed from 632-inject-log-in-search to develop July 11, 2024 15:38
@yongbeomkwak yongbeomkwak merged commit bd353fa into develop Jul 12, 2024
3 checks passed
@yongbeomkwak yongbeomkwak deleted the 765-fix-playlist-detail-layout-mistake branch July 12, 2024 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@baekteun baekteun baekteun left review comments

@KangTaeHoon KangTaeHoon KangTaeHoon approved these changes

@youn9k youn9k youn9k approved these changes

@kimdaehee0824 kimdaehee0824 Awaiting requested review from kimdaehee0824 kimdaehee0824 is a code owner

Assignees

@yongbeomkwak yongbeomkwak

Labels
0️⃣ Priority: Critical 우선순위 긴급
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

playlistDetail 디자인 검수
4 participants
@yongbeomkwak @baekteun @KangTaeHoon @youn9k