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[#6, #7] : CoreDataManager 추가, SearchingPlayListView와 Swipe Cell 구현 #10

Merged
merged 9 commits into from
Jun 13, 2023

Conversation

yeniful
Copy link
Contributor

@yeniful yeniful commented Jun 10, 2023

@Juhwa-Lee1023
@LeeSungNo-ian
@2youngjun


CoreDataManager 추가, SearchingPlayListView와 Swipe Cell 구현


OutLine

Work Contents

  • SwiftUI에 맞게 CoreDataManager를 추가했습니다. 메서드 명만으로 이해하기 쉽도록 기존 v1의 메서드 명에서 기능에 맞게 이름을 변경했습니다. (주석 참고)
  • SwiftUI의 List를 활용하여 구현했던 Cell들을 Custom Cell로 변경했습니다.
  • 찾은 음악들(현재는 목업 데이터)을 App Storage에 저장해서 보존하는 기능을 추가했습니다.
  • 저장하기 Alert를 추가했습니다.
RPReplay_Final1686417473.MP4

To Reviewers

  • 데이터 추가, 삭제시 UI가 매끄럽지 않은 문제가 있습니다. 다음 PR 때 수정하도록 하겠습니다.
  • SwipeAction Button을 많이 당겼을 때 애니메이션과 함께 삭제하는 기능 추가가 필요합니다. 추후 업데이트 하겠습니다.
  • 샤잠 로직을 추가하면 필요에 따라 CodeData 메서드 로직을 바꾸면 될 것 같습니다.
  • 작업이 늦어져 죄송합니다 🥲

Close #6

Copy link
Member

@Juhwa-Lee1023 Juhwa-Lee1023 left a comment

Choose a reason for hiding this comment

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

고생하셧습니다!!

.gesture(DragGesture(minimumDistance: 15, coordinateSpace: .local)
.onChanged({ (value) in
let totalSlide = value.translation.width + oldOffset
if (Int(minTrailingOffset)...0 ~= Int(totalSlide)) {
Copy link
Member

Choose a reason for hiding this comment

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

범위안에 있는지를 확인하는 로직을 이렇게 처리할 수도 있군요! 배워갑니다 ㅎㅎ

@yeniful
Copy link
Contributor Author

yeniful commented Jun 12, 2023

@Juhwa-Lee1023 회의 때 리뷰 주신대로 파일 분리 완료했습니다. 저장된 PlayList에서도 사용할 수 있도록 SearchingPlayListViewPlayListView로 수정해서 ReusableUI로 옮겼습니다.

@Juhwa-Lee1023 Juhwa-Lee1023 merged commit cb76cd7 into develop Jun 13, 2023
@Juhwa-Lee1023 Juhwa-Lee1023 deleted the feat/Data branch June 13, 2023 11:59
@yeniful yeniful self-assigned this Jun 14, 2023
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.

[Feat] : SavePlayListView
2 participants