Fix #565: Domain on-boarding flag

app/src/main/AndroidManifest.xml

@@ -62,6 +62,10 @@
     <activity
       android:name=".settings.profile.ProfileResetPinActivity"
       android:screenOrientation="portrait" />
+    <activity
+      android:name=".splash.OnboardingActivity"
+      android:screenOrientation="portrait"
+      android:theme="@style/OppiaThemeWithoutActionBar" />
     <activity
       android:name=".splash.SplashActivity"
       android:screenOrientation="portrait"

app/src/main/java/org/oppia/app/activity/ActivityComponent.kt

@@ -18,6 +18,8 @@ import org.oppia.app.settings.profile.ProfileEditActivity
 import org.oppia.app.settings.profile.ProfileListActivity
 import org.oppia.app.settings.profile.ProfileRenameActivity
 import org.oppia.app.settings.profile.ProfileResetPinActivity
+import org.oppia.app.splash.OnboardingActivity
+import org.oppia.app.splash.SplashActivity
 import org.oppia.app.story.StoryActivity
 import org.oppia.app.story.testing.StoryFragmentTestActivity
 import org.oppia.app.testing.AudioFragmentTestActivity
@@ -66,13 +68,15 @@ interface ActivityComponent {
   fun inject(htmlParserTestActivity: HtmlParserTestActivity)
   fun inject(myDownloadsActivity: MyDownloadsActivity)
   fun inject(navigationDrawerTestActivity: NavigationDrawerTestActivity)
+  fun inject(onboardingActivity: OnboardingActivity)
   fun inject(pinPasswordActivity: PinPasswordActivity)
   fun inject(profileActivity: ProfileActivity)
   fun inject(questionPlayerActivity: QuestionPlayerActivity)
   fun inject(profileEditActivity: ProfileEditActivity)
   fun inject(profileListActivity: ProfileListActivity)
   fun inject(profileRenameActivity: ProfileRenameActivity)
   fun inject(profileResetPinActivity: ProfileResetPinActivity)
+  fun inject(splashActivity: SplashActivity)
   fun inject(storyActivity: StoryActivity)
   fun inject(topicActivity: TopicActivity)
   fun inject(topicTestActivity: TopicTestActivity)

app/src/main/java/org/oppia/app/home/UserAppHistoryViewModel.kt

@@ -8,4 +8,7 @@ class UserAppHistoryViewModel : HomeItemViewModel() {
   var isAppAlreadyOpened = ObservableField<Boolean>(false)
 
   fun setAlreadyAppOpened(alreadyOpenedApp: Boolean) = isAppAlreadyOpened.set(alreadyOpenedApp)
+  var isAppAlreadyOnboarded = ObservableField<Boolean>(false)
+
+  fun setAlreadyAppOnboarded(isalreadyOnboardedApp: Boolean) = isAppAlreadyOnboarded.set(isalreadyOnboardedApp)
 }

app/src/main/java/org/oppia/app/splash/OnboardingActivity.kt

@@ -0,0 +1,21 @@
+package org.oppia.app.splash
+
+import android.os.Bundle
+import org.oppia.app.activity.InjectableAppCompatActivity
+import org.oppia.app.help.OnboardingActivityPresenter
+import org.oppia.app.profile.ProfileActivity
+import javax.inject.Inject
+
+/** An activity that shows a temporary Onboarding page until the app is onborarded then navigates to [ProfileActivity]. */
+class OnboardingActivity : InjectableAppCompatActivity() {
+
+  @Inject
+  lateinit var onboardingActivityPresenter: OnboardingActivityPresenter
+
+  override fun onCreate(savedInstanceState: Bundle?) {
+    super.onCreate(savedInstanceState)
+    activityComponent.inject(this)
+    onboardingActivityPresenter.handleOnCreate()
+  }
+}
+

app/src/main/java/org/oppia/app/splash/OnboardingActivityPresenter.kt

@@ -0,0 +1,30 @@
+package org.oppia.app.help
+
+import android.content.Intent
+import androidx.appcompat.app.AppCompatActivity
+import kotlinx.android.synthetic.main.dummy_onboard.*
+import org.oppia.app.R
+import org.oppia.app.activity.ActivityScope
+import org.oppia.app.profile.ProfileActivity
+import org.oppia.domain.OnboardingFlowController
+import javax.inject.Inject
+
+/** The presenter for [OnboardingActivity]. */
+@ActivityScope
+class OnboardingActivityPresenter @Inject constructor(
+  private val activity: AppCompatActivity,
+  private val onboardingFlowController: OnboardingFlowController
+) {
+
+  fun handleOnCreate() {
+    activity.setContentView(R.layout.dummy_onboard)
+    activity.onboard.setOnClickListener({ subscribeToUserAppHistory() })
+  }
+
+  private fun subscribeToUserAppHistory() {
+    onboardingFlowController.markOnboardingFlowCompleted()
+    val intent = Intent(activity, ProfileActivity::class.java)
+    activity.startActivity(intent)
+    activity.finish()
+  }
+}

app/src/main/java/org/oppia/app/splash/SplashActivity.kt

@@ -1,22 +1,21 @@
 package org.oppia.app.splash
 
-import android.content.Intent
 import android.os.Bundle
-import android.view.WindowManager
-import androidx.appcompat.app.AppCompatActivity
-import org.oppia.app.R
+import org.oppia.app.activity.InjectableAppCompatActivity
+import org.oppia.app.help.SplashActivityPresenter
 import org.oppia.app.profile.ProfileActivity
+import javax.inject.Inject
 
 /** An activity that shows a temporary loading page until the app is fully loaded then navigates to [ProfileActivity]. */
-class SplashActivity : AppCompatActivity() {
+class SplashActivity : InjectableAppCompatActivity() {
+
+  @Inject
+  lateinit var splashActivityPresenter: SplashActivityPresenter
 
   override fun onCreate(savedInstanceState: Bundle?) {
     super.onCreate(savedInstanceState)
-    setContentView(R.layout.splash_activity)
-
-    window.setFlags(WindowManager.LayoutParams.FLAG_FULLSCREEN, WindowManager.LayoutParams.FLAG_FULLSCREEN);
-    val intent = Intent(this@SplashActivity, ProfileActivity::class.java)
-    startActivity(intent)
-    finish()
+    activityComponent.inject(this)
+    splashActivityPresenter.handleOnCreate()
   }
+
 }

app/src/main/java/org/oppia/app/splash/SplashActivityPresenter.kt

@@ -0,0 +1,61 @@
+package org.oppia.app.help
+
+import android.content.Intent
+import android.view.WindowManager
+import androidx.appcompat.app.AppCompatActivity
+import androidx.lifecycle.LiveData
+import androidx.lifecycle.Observer
+import androidx.lifecycle.Transformations
+import org.oppia.app.R
+import org.oppia.app.activity.ActivityScope
+import org.oppia.app.model.OnboardingFlow
+import org.oppia.app.profile.ProfileActivity
+import org.oppia.app.splash.OnboardingActivity
+import org.oppia.domain.OnboardingFlowController
+import org.oppia.util.data.AsyncResult
+import javax.inject.Inject
+
+/** The presenter for [OnboardingActivity]. */
+@ActivityScope
+class SplashActivityPresenter @Inject constructor(
+  private val activity: AppCompatActivity,
+  onboardingFlowController: OnboardingFlowController
+) {
+
+  val sample = onboardingFlowController.getOnboardingFlow()
+
+  fun handleOnCreate() {
+    activity.setContentView(R.layout.splash_activity)
+    activity.window.setFlags(WindowManager.LayoutParams.FLAG_FULLSCREEN, WindowManager.LayoutParams.FLAG_FULLSCREEN);
+    subscribeToOnboardingFlow()
+  }
+
+  private fun showOnboardingActivity() {
+    val intent = Intent(activity, OnboardingActivity::class.java)
+    activity.startActivity(intent)
+    activity.finish()
+  }
+
+  private fun subscribeToOnboardingFlow() {
+    getOnboardingFlow().observe(activity, Observer<OnboardingFlow> { result ->
+      if (result.alreadyOnBoardedApp) {
+        val intent = Intent(activity, ProfileActivity::class.java)
+        activity.startActivity(intent)
+        activity.finish()
+      } else {
+        showOnboardingActivity()
+      }
+    })
+  }
+
+  private fun getOnboardingFlow(): LiveData<OnboardingFlow> {
+    // If there's an error loading the data, assume the default.
+    return Transformations.map(sample, ::processOnboardingFlowResult)
+  }
+
+  private fun processOnboardingFlowResult(appHistoryResult: AsyncResult<OnboardingFlow>): OnboardingFlow {
+    if (appHistoryResult.isFailure()) {
+    }
+    return appHistoryResult.getOrDefault(OnboardingFlow.getDefaultInstance())
+  }
+}

app/src/main/res/layout/dummy_onboard.xml

@@ -0,0 +1,14 @@
+<?xml version="1.0" encoding="utf-8"?>
+<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
+  android:layout_width="match_parent"
+  android:layout_height="match_parent"
+  android:gravity="center"
+  android:orientation="vertical">
+
+  <Button
+    android:id="@+id/onboard"
+    android:layout_width="wrap_content"
+    android:layout_height="wrap_content"
+    android:text="onboard" />
+</LinearLayout>
+