Skip to content

Conversation

@YangHJ2415
Copy link
Collaborator

@YangHJ2415 YangHJ2415 commented Oct 10, 2025

close #65

Service를
파싱 역할을 하는 Parser와 / 핵심 로직의 Core와 / Core를 호출하는 Usecase 인터페이스로 구분해
코드의 양을 줄이고, 역할을 분할 하였습니다.

캐시 관련해 개선 작업이 들어갔습니다.

@YangHJ2415
Copy link
Collaborator Author

YangHJ2415 commented Oct 10, 2025

브런치 문제인건지,🤔
back\koreaTravelGuide\common\config\RedisConfig.kt
back\koreaTravelGuide\KoreaTravelGuideApplication.kt
두 군데에서 충돌이 한번씩 있어 처리 후 rebase 했습니다.

@JIWONKIMS
Copy link
Collaborator

이 PR은 현준님 설명이 필요한 부분이 많아서 당장 승인하지는 않겠습니다
RedisConfig, KoreaTravelGuideApplication, ChatRoomController, ChatMessageController은 현준님 코드가 아닐텐데 상의 없이 코드 수정이 들어갔고
다른 현준님 코드들도 수정한 부분이 많아서 당장 승인하기는 어렵습니다

@YangHJ2415
Copy link
Collaborator Author

언급하신 RedisConfig, KoreaTravelGuideApplication, ChatRoomController, ChatMessageController
모두 제가 pull 받고 리베이스 하는 과정에서 뭔가 꼬인건지 수정이 들어간거 같은데,
(fix: RedisConfig merge 충돌 커밋 내용이 이에 해당 합니다.)

깔끔하게 브런치 새로 파서
제가 수정한것만 적용해서 새로 올려보도록 하겠습니다.

@YangHJ2415 YangHJ2415 closed this Oct 11, 2025
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.

Tour 도메인 리팩토링

3 participants