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

DiaryEditViewController 구성 / Location 흐름 구축 #176

Merged
merged 6 commits into from
Dec 8, 2022

Conversation

LEEYOONJONG
Copy link
Member

12/7 #166 #168 #171

작업한 내용

imagePicker를 통해 갤러리의 이미지를 선택하고, 뷰에 반영될 수 있도록 했습니다.

태그 추가 시 적절한 UI 형태로 반환하도록 했습니다.

  • 이 과정에서 텍스트필드를 새롭게 작성했으며, 관련하여 키보드 가림 이슈를 다시 수정했습니다.

위치 버튼을 눌렀을 때 레포지토리를 통해 위치를 가져오는 일련의 흐름을 구축했습니다.

  • 뷰컨트롤러를 제공할 필요가 없어, session이 아닌 일반적인 클린 아키텍처 흐름으로 구성했습니다.

image

Copy link
Collaborator

@rudah7 rudah7 left a comment

Choose a reason for hiding this comment

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

와우.. 코드 수정 440줄! ㄷㄷ..

수고하셨습니다!

debugPrint("위치 서비스 on")
self.locationManager.startUpdatingLocation()
} else {
print("위치 서비스 off 상태")
Copy link
Collaborator

Choose a reason for hiding this comment

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

그냥 보다가 궁금해진건데, 다른 부분들은 debugPrint() 쓰시다 여기는 print()군요..?!

Copy link
Member Author

Choose a reason for hiding this comment

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

ㅋ.ㅋ 이런,,,,,

Comment on lines +14 to +15
var locationSubject: PublishSubject<Location> { get set }
var addressSubject: PublishSubject<String> { get set }
Copy link
Collaborator

Choose a reason for hiding this comment

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

subject에 대한 고민이 정말.. 사람 미치게 하죠ㅎㅎㅋㅋ 이 주제에 대해서 오늘내일 이야기를 나눠 보고, 이번 주 칼럼에 실어 봅시다.

debugPrint("tags : ", tags)
// TODO: 이미지 업로드 해야
debugPrint("bodyText : ", bodyTextView.text)
// TODO: MusicInfo엔 어떤 정보가 string으로?
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 부분이 아마 "오늘 할 일"의 카테고리를 달고 넘어가게 될 것 같은데, 머지하기 전에 이야기를 나눠봅시다.

@radiantchoi radiantchoi merged commit aa18604 into develop Dec 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants