firebase / quickstart-android

Firebase Quickstart Samples for Android
https://firebase.google.com
Apache License 2.0
8.87k stars 7.32k forks source link

[Internal] Translate mlkit app to kotlin #635

Closed samtstern closed 6 years ago

samtstern commented 6 years ago

See #557

thatfiredev commented 6 years ago

I'll be handling this one.

samtstern commented 6 years ago

I just started looking at firestore and I think the process I used for inappmessaging was probably too simple for the more complex quickstarts.

I think maybe instead of Foo.java and KotlinFoo.kt in the same package, I want to make a .java and a .kt package at the root of the quickstart and then have identical class names in each subtree.

thatfiredev commented 6 years ago

I understand. I'll do the same for this quickstart.

samtstern commented 6 years ago

Or if you come up with something better, feel free to experiment!