Skip to content

[Feature/#21] : 코드 리팩토링#22

Merged
gaeulzzang merged 7 commits intodevelopfrom
feature/#21-total-refactor
Feb 18, 2025
Merged

[Feature/#21] : 코드 리팩토링#22
gaeulzzang merged 7 commits intodevelopfrom
feature/#21-total-refactor

Conversation

@gaeulzzang
Copy link
Contributor

✅ 𝗖𝗵𝗲𝗰𝗸-𝗟𝗶𝘀𝘁

  • merge할 브랜치의 위치를 확인해 주세요.(main❌/develop⭕)
  • 리뷰가 필요한 경우 리뷰어를 지정해 주세요.
  • 리뷰는 PR이 올라오면 최대한 빠르게 진행합니다.
  • P1 단계의 리뷰는 빠르게 확인 후 반영합니다.
  • Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다.

📌 𝗜𝘀𝘀𝘂𝗲𝘀

📎 𝗪𝗼𝗿𝗸 𝗗𝗲𝘀𝗰𝗿𝗶𝗽𝘁𝗶𝗼𝗻

  • id Long 타입 -> Int 타입으로 수정
  • 불필요한 import문, 리소스 제거
  • 지도 화면에서 바텀 시트 SideEffect로 분리

📷 𝗦𝗰𝗿𝗲𝗲𝗻𝘀𝗵𝗼𝘁

UI 변동 사항이 없어 생략하겠습니다

💬 𝗧𝗼 𝗥𝗲𝘃𝗶𝗲𝘄𝗲𝗿𝘀

영서야 캘린더 수정 드가자

@gaeulzzang gaeulzzang self-assigned this Feb 18, 2025
@gaeulzzang gaeulzzang requested a review from a team as a code owner February 18, 2025 14:18
Copy link
Contributor

@jjwm10625 jjwm10625 left a comment

Choose a reason for hiding this comment

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

수고하셨어용 이제 캘린더 수정 드갑니다 👊

@gaeulzzang gaeulzzang merged commit c3c09bd into develop Feb 18, 2025
1 check passed
Copy link
Contributor

@nhyeonii nhyeonii left a comment

Choose a reason for hiding this comment

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

코드 확인 완료 ~~ 🤩 진짜 진짜 수고 많았습니동 .. 🫰 고생했다 !!!!!!!!!!!!!!!

@gaeulzzang gaeulzzang deleted the feature/#21-total-refactor branch February 20, 2025 07:07
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.

[REFACTOR] : 코드 리팩토링

3 participants