architecture-samples icon indicating copy to clipboard operation
architecture-samples copied to clipboard

Architecture Sample Proposal with Koin

Open arnaudgiuliani opened this issue 4 years ago • 2 comments
trafficstars

Hello!

I would like to propose a version of the app, done with Koin. A branch is almost ready here: https://github.com/InsertKoinIO/architecture-samples/tree/dev-koin

Are you interested? What is the process to integrate it?

Thanks.

arnaudgiuliani avatar May 27 '21 12:05 arnaudgiuliani

Hey Arnaud! That looks great!

The project is meant to be used a base for other repos, such as the one you linked. We're not linking to external repos anymore because they tend to get outdated quickly (as our own does!) and we can't review all of them. We're moving towards a single version of the app using all the recommended tools in the architecture guidance.

Btw there's an error in the README: The ServiceLocator class is removed. Object creation and scoping is handled by Hilt.

Also, I'd link to the "compare" view of the dev-hilt vs dev-koin in the README or Koin's documentation, which is useful to show a migration path: https://github.com/android/architecture-samples/compare/dev-hilt...InsertKoinIO:dev-koin

Good job!

JoseAlcerreca avatar Jun 01 '21 07:06 JoseAlcerreca

Hey Jose,

The project is meant to be used a base for other repos, such as the one you linked. We're not linking to external repos anymore because they tend to get outdated quickly (as our own does!) and we can't review all of them. We're moving towards a single version of the app using all the recommended tools in the architecture guidance.

Yep, if you're interested having a Koin part. Let me know I can help.

Btw there's an error in the README: The ServiceLocator class is removed. Object creation and scoping is handled by Hilt.

Yep, perhaps still some typos.

Also, I'd link to the "compare" view of the dev-hilt vs dev-koin in the README or Koin's documentation, which is useful to show a migration path: dev-hilt...InsertKoinIO:dev-koin

Do you want a more descriptive README file here?

Thanks.

arnaudgiuliani avatar Jun 08 '21 07:06 arnaudgiuliani