robincaroff/workshop-kmp

Text repository for the Kotlin Multiplatform Workshop

Hi there 👋

👨‍💻📲 Mobile technology enthusiast and developer (Android and iOS native).

💫 Mobile Lead at U IRIS.

📣 Speaker and co-organizer at GDG Nantes Android and FRAUG 🐸 (French Android User Group).

💬 Reach me on

• Twitter @RobinCaroff
• LinkedIn
• SpeakerDeck

workshop-kmp's Issues

Step 3 : typo to => two

Notice that we create two methods with the same purpose.

xyz.mlumeau.kosmos.kore.usecases.GetAPOD package

In step 6, GetAPOD file created with wrong package (missing kore)

README: Mettre à part les instructions de la présentation

Les instructions de compilation du WS devraient être mises à part pour ne garder dans le README qu'une présentation du WS.

Step 6 : Inutile de recréer APODRepositoryRemote déjà créé à une step précédente

Now create the remote repository interface : .../kore/data/APODRepositoryRemote.kt

package xyz.mlumeau.kosmos.kore.data

import xyz.mlumeau.kosmos.kore.model.APOD

interface APODRepositoryRemote {
suspend fun getAPOD(): APOD?
fun getAPOD(completion: (APOD) -> Unit, failure: () -> Unit)
}
and the implementation : .../kore/data/APODRepositoryRemoteImpl.kt

package xyz.mlumeau.kosmos.kore.data

import xyz.mlumeau.kosmos.kore.model.APOD
import xyz.mlumeau.kosmos.kore.NasaAPIRemote
import xyz.mlumeau.kosmos.kore.requestAPOD
import xyz.mlumeau.kosmos.kore.service.nasa.NasaApi

class APODRepositoryRemoteImpl : APODRepositoryRemote {
private val nasaAPI: NasaApi = NasaAPIRemote()

override suspend fun getAPOD() = nasaAPI.getAPOD()

override fun getAPOD(completion: (APOD) -> Unit, failure: () -> Unit) {
    requestAPOD(this, completion, failure)
}

}

Suggestion : Aligner le nom des step et les numéros(regrouper 1,2,3 dans StepOne)?

Report a mistake devrait amener directement à la page des issues

Report a mistake devrait amener directement à la page des issues du workshop.