#29: AuthActivity를 만들어 회원가입을 진행해요 #34
Conversation
flash159483
added
🍩브라우니🍩
| @Module | ||
| @InstallIn(SingletonComponent::class) | ||
| interface NavigationModule { | ||
| @Binds | ||
| @Singleton | ||
| fun provideNavigator(navigator: NavigatorImpl): Navigator | ||
| } |
There was a problem hiding this comment.
오호~ 이거 동작하는거죠 ?
제가 알기로 abstract class만 @BINDS 지원해주는 줄 알았는데, interface도 가능하군요
There was a problem hiding this comment.
binds는 interface 아니면 abstract class에 동작해요
| inline fun <reified T : Activity> Context.buildIntent( | ||
| vararg argument: Pair<String, Any?>, | ||
| ) = Intent(this, T::class.java).apply { | ||
| putExtras(bundleOf(*argument)) | ||
| } |
There was a problem hiding this comment.
이거 Navigation Module에서만 사용되는거 아닐까요 ??
그렇다면, navigation Module내에서 Extensions으로 사용할 수 있을 것 같아요
There was a problem hiding this comment.
common에서는 Extensions가 많이 사용될 수 있어서
extensions 패키지 하나 만들고, 내부에 세부 파일 만들어서 사용하시면 더 좋지 않을까요 ?
There was a problem hiding this comment.
저는 extensions는 대부분 저 extensions 파일에 넣어서 만약 분리가 필요하면 그때 가서 하면 될듯하네요
There was a problem hiding this comment.
이거 Navigation Module에서만 사용되는거 아닐까요 ??
그렇다면, navigation Module내에서 Extensions으로 사용할 수 있을 것 같아요
이 코멘트도 확인부탁해용
There was a problem hiding this comment.
extension을 너무 흩어놓는거 같고 다른 곳에서 쓸수도 있어서 common에 넣어놨어요
flash159483
force-pushed
the
feature/flash159483/#29
branch
from
1bf0f81 to
4c118b0
Compare
1. 📄 관련된 이슈 및 소개
#29
2. 🔥 변경된 점
AuthActivity를 추가하고 로직을 옮겨요
Activity 끼리 움직일 수 있는 Navigator를 추가해요
3. ✅ 필수 체크 사항
4. 📸 작업물 사진 공유(선택)
5. 💡알게된 혹은 궁금한 사항