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.

Sign up for GitHub

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

Jump to bottom

[ADD] Navigator 추가 #336

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from
Open

[ADD] Navigator 추가 #336

wants to merge 7 commits into from

Conversation

dongx0915
Copy link
Member

@dongx0915 dongx0915 commented Feb 22, 2024 •
edited
Loading

📌 개요

✨ 작업 내용

  • Navigator를 이용하여 화면 이동하도록 수정 (멀티 모듈 리팩토링 사전 작업)
  • Navigator를 이용한 화면 이동 예시 코드 추가 (프로필 -> 코스 상세)

✨ PR 포인트

  • 멀티 모듈로 구성 시 A 액티비티에서 B 액티비티를 참조할 수 없어지므로 A -> B로 이동시 필요한 Extra를 B 액티비티 내에 선언할 수 없어집니다.
  • 이를 해결하기 위해 화면 이동시 전달하는 Extra Key 값을 Navigator 내에 선언하도록 수정 해봤는데 의견 부탁드립니다.(CourseDetailNavigator 참고)

📸 스크린샷/동영상

@dongx0915 dongx0915 added ADD ➕ feat 이외의 부수적인 코드, 파일, 라이브러리 추가 동현 🐨 동현 담당 labels Feb 22, 2024
@dongx0915 dongx0915 self-assigned this Feb 22, 2024
unam98
unam98 approved these changes Feb 25, 2024
View reviewed changes
Copy link
Contributor

@unam98 unam98 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! EXTRA 상수는 companion object 안에 const val로 만들 거면 아예 별도로 ExtraUtils 같은 걸 하나 만들어서 한 곳에서 관리해도 좋겠다는 생각이 들었습니다!

leeeha
leeeha approved these changes Feb 25, 2024
View reviewed changes
Copy link
Member

@leeeha leeeha left a comment •
edited
Loading

Choose a reason for hiding this comment

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

수고하셨습니다!
ExtraUtils 같은 걸 따로 만드는 것도 괜찮은 방법인 거 같아요! 만약 만든다면 navigator 모듈에 만들게 되나요??

@unam98 unam98 force-pushed the develop branch 3 times, most recently from fb1f98f to f8525c2 Compare June 19, 2024 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@unam98 unam98 unam98 approved these changes

@leeeha leeeha leeeha approved these changes

@sxunea sxunea Awaiting requested review from sxunea

Assignees

@dongx0915 dongx0915

Labels
ADD ➕ feat 이외의 부수적인 코드, 파일, 라이브러리 추가 동현 🐨 동현 담당
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[ADD] Navigator 추가
3 participants
@dongx0915 @leeeha @unam98