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] #123 플레이리스트 #124

Merged
merged 18 commits into from
Jun 30, 2023
Merged

[Feat] #123 플레이리스트 #124

merged 18 commits into from
Jun 30, 2023

Conversation

yeniful
Copy link
Contributor

@yeniful yeniful commented Jun 28, 2023

Issue

  • 123

Key Changes

  • 플레이리스트 기능 추가
  • 추가, 삭제, 순서 변경
  • 플레이리스트에서 노래 직접 재생 가능
RPReplay_Final1687991094.MP4

To Reviewers

  • 순서를 지정해주기 위해 CollectedSong에 order라는 프로퍼티를 추가했습니다. 원하는 기준으로 정렬시 사용해주세요.
  • persistence에 있는 fetchedrequest 로직을 덜어냈습니다. fetchedreques는 view에서 사용되어야하기 때문에 view에서 parameter로 전달하는 방식으로 수정했습니다.
  • SongInfo와 Song을 합쳐서 parameter를 줄여야할 때가 오지 않았나 싶습니다.
  • 팀 선택이 AppStorage로 저장되어서, 플레이리스트 화면에서 현재 곡과 다른 팀의 노래를 틀었을 때 메인으로 가는 문제가 있습니다. Binding으로 추가하려고 했으나 기본 배경이 selectedTeam을 기준으로 바뀌기 때문에, 구조를 바꾸는 방법을 고안해야할 것 같습니다.

(Optional) References

  • 참고한 레퍼런스가 있다면 링크를 추가해주세요.

@Anti9uA Anti9uA merged commit 69924ae into Develop Jun 30, 2023
@yeniful yeniful deleted the PlayListViewRestore branch July 4, 2023 17:02
@JIWON1923 JIWON1923 restored the PlayListViewRestore branch October 16, 2023 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants