Hey guys, I am I doing this wrongly or? I have a s...

# koin

[Nikola Milovic]

Hey guys, I am I doing this wrongly or? I have a single activity approach with multi feature modules with their fragments and VM's. Now I have a DI package in my Module A with repository, network and VM koin modules. This is what I am doing at the moment. I just reference those 3 modules from the other package.

open class App : Application() {

    override fun onCreate() {
        super.onCreate()
        startKoin {
            // Android context
            androidContext(this@App)
            // modules
            modules(com.example.core.di.networkModule, (com.example.homeFeed.di.)viewModelModule, (com.example.homeFeed.di.)repositoryModule, com.example.homeFeed.di.networkModule)
        }
    }

}

But I crash with this as the output https://hastebin.com/qizixidumu.sql
Also is there a better way to handle koin modules in a multimodule app? Like having an array of modules and then just loading them when needed as the fragments change? Thank you!

[Allan Wang]

Just to confirm, did you add your App to the manifest? Adding modules looks correct. This is also running on a device, and not a test correct?

[Nikola Milovic]

@Allan Wang I did yes, it has worked on the Core module, but then I added the HomeFeed module and it started crashing. But in theory it should work? If thats the case I'll go hunting for the cause

[Nikola Milovic]

It crashes once I navigate to the new Module

[Allan Wang]

I don’t see how adding homefeed should stop it from recognizing the koin application. If it’s missing a dependency that’s different though

[Allan Wang]

what does “navigating to the new module” mean

[Nikola Milovic]

@Allan Wang Single activity, go to a fragment in another module

[Allan Wang]

is network in a separate module as well? Did you end up adding a new android module when adding the new koin module?

[Allan Wang]

If you’re not sure you can try building the app and checking the manifest just to make sure the application name (referencing .App) still exists

[Nikola Milovic]

@Allan Wang Thank you! That was it. The .App was missing