Skip to content

Conversation

@513sojin
Copy link
Collaborator

@513sojin 513sojin commented Nov 16, 2023

🌱 작업한 내용

  • 버그 수정(지도에서 선택 했을때 마커가 안찍히는 경우)
  • 바텀시트 연결 및 버튼 연결 코드 수정
  • 컴바인 코코아 공식문서

🌱 PR Point

  • 바텀시트에서 버튼 연결하는 부분 combine cocoa를 사용했습니다

📮 관련 이슈

Comment on lines -29 to -33
var completeButtonTapped: Driver<Void> {
completeButton.publisher(for: .touchUpInside)
.map { _ in }
.asDriver()
}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

버튼에 publisher를 연결하는 과정을 combineCocoa를 사용하는 것으로 수정했습니다

Comment on lines +247 to +251
self.completeButton.tapPublisher.sink { [weak self] _ in
guard let self = self else { return }
guard let text = self.bottomSheetTextField.text else { return }
self.completeButtonTapAction?(text)
}.store(in: cancelBag)
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

완성하기 버튼이 눌린 경우는 textfield의 내용을 클로저에 전달해주었습니다


struct CourseDrawingRequestData: Codable {
let path: [RNLocationModel]
// let title: String
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

서버 배포되면 title 관련 주석 삭제하도록 하겠습니당

Copy link
Collaborator

Choose a reason for hiding this comment

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

@513sojin 513sojin self-assigned this Nov 16, 2023
@513sojin 513sojin added Feat 새로운 기능 구현 Fix 소진🍊 labels Nov 16, 2023
@513sojin 513sojin requested a review from thingineeer November 16, 2023 11:37
Copy link
Collaborator

@thingineeer thingineeer left a comment

Choose a reason for hiding this comment

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

수고 하셨습니다~

let departureLocationModel = addressList[indexPath.item].toDepartureLocationModel()
courseDrawingVC.setData(model: departureLocationModel)
SelectedInfo.shared.type = .other
courseDrawingVC.setData(model: departureLocationModel)
Copy link
Collaborator

Choose a reason for hiding this comment

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

순서의 중요성

private var BottomsheetType: SheetType!

var backgroundTapAction: (() -> Void)?
var completeButtonTapAction: ((String) -> Void)?
Copy link
Collaborator

Choose a reason for hiding this comment

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

굿굿 감사합니다


struct CourseDrawingRequestData: Codable {
let path: [RNLocationModel]
// let title: String
Copy link
Collaborator

Choose a reason for hiding this comment

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

@513sojin 513sojin merged commit 7cac6bc into Runnect:develop Nov 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feat 새로운 기능 구현 Fix 소진🍊

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Add] #208 - 코스 업로드 api 수정

2 participants