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

🔀 :: (#812) Music Detail Feature 디렉토리 정리 #906

Merged
merged 3 commits into from
Jul 30, 2024

Conversation

baekteun
Copy link
Member

💡 배경 및 개요

MusicDetailFeature의 디렉토리를 정리했어요.

📃 작업내용

기능 기반으로 디렉토리 구조에서 도메인 기반으로 디렉토리 구조를 변경했어요
기능 기반:
ViewControllers
- MusicDetailViewController.swift
- KaraokeViewController.swift
Views
- MusicDetailView.swift
- MusicControlView.swift
- KaraokeNumberView.swift

도메인 기반:
MusicDetail
- MusicDetailViewController.swift
- Views
- MusicDetailView.swift
- MusicControlView.swift
Karaoke
- KaraokeNumberViewController.swift
- Views
- KaraokeNumberView.swift

✅ PR 체크리스트

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

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

Copy link

github-actions bot commented Jul 29, 2024

✅ Successful finished SwiftLint

@github-actions github-actions bot added the 1️⃣ Priority: High 우선순위 상 label Jul 29, 2024
Copy link

✅ 이슈와 PR의 Labels 동기화를 성공했어요!

@baekteun baekteun force-pushed the 812-credit-worker-song-list branch 2 times, most recently from 49acc72 to 3ca284a Compare July 29, 2024 14:38
@baekteun baekteun force-pushed the 812-credit-worker-song-list branch from 3ca284a to 425e32e Compare July 29, 2024 17:16
@baekteun baekteun merged commit af3c5a3 into develop Jul 30, 2024
3 checks passed
@baekteun baekteun deleted the 812-credit-worker-song-list branch July 30, 2024 00:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1️⃣ Priority: High 우선순위 상
Projects
None yet
Development

Successfully merging this pull request may close these issues.

크레딧 작업자가 한 노래 리스트 페이지 작업
3 participants