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

[iOS] 여정 기록 중 지도 위에 Spot 등록 로직 구현 #289

Merged
merged 6 commits into from
Dec 10, 2023

Conversation

yoondj98
Copy link
Collaborator

❗ 배경

작업 배경에 대한 설명을 작성합니다.
Issue에 대한 링크를 첨부합니다.

🔧 작업 내역

작업한 내용들을 나열합니다.
간결하게 리스트 업하고, 자세한 설명은 아래 리뷰 노트에서 합니다.

  • 여정 기록 중 Spot을 등록 시 지도 위에 실시간으로 나타나도록 구현합니다.
  • Spot 생성 후 뒤로가기 클릭 시 Spot이 모두 안보이도록 구현했습니다.

🧪 테스트 방법

동작을 테스트할 수 있는 방법을 설명합니다.
앱 실행 방법일 수 있고, 유닛 테스트 실행 방법일 수 있습니다.

📝 리뷰 노트

작업 내역에 대한 자세한 설명을 작성합니다.

  • Coordinator를 활용하여 SpotSaveViewController에서 NaviagteMapViewController로 Spot을 넘겨주었습니다.
  • 데이터가 안넘겨지는 이슈가 있었는데 UserDefaults 내 값이 nil로 설정되는 코드로 인해 시간이 오래 걸렸습니다.

📸 스크린샷

작업한 내용에 대한 스크린샷, 영상 등을 첨부합니다.

@yoondj98 yoondj98 added the ✨ 신규 기능 신규 기능 개발 label Dec 10, 2023
@yoondj98 yoondj98 added this to the 🏃‍♂️ 여정 기록 milestone Dec 10, 2023
@yoondj98 yoondj98 self-assigned this Dec 10, 2023
@SwiftyJunnos SwiftyJunnos changed the base branch from main to iOS/release December 10, 2023 17:33
Copy link
Member

@PushedGun PushedGun left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

func popToHomeWithSpot(from coordinator: Coordinator, spot: Spot) {
// 미사용 함수
}

Copy link
Member

Choose a reason for hiding this comment

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

이건 구현 예정이신가요?

@SwiftyJunnos SwiftyJunnos merged commit 3e1e650 into iOS/release Dec 10, 2023
33 checks passed
@SwiftyJunnos SwiftyJunnos deleted the iOS/task/drawing-spot-in-recording branch December 10, 2023 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ 신규 기능 신규 기능 개발
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants