[All] Add missing pandata events (#3629)

hermannakos · web-flow · commit 10ba8abe77ab · 2026-04-07T15:09:59.000+02:00
diff --git a/apps/student/src/main/java/com/instructure/student/features/dashboard/compose/NewDashboardFragment.kt b/apps/student/src/main/java/com/instructure/student/features/dashboard/compose/NewDashboardFragment.kt
@@ -22,6 +22,7 @@ import android.view.View
 import android.view.ViewGroup
 import androidx.compose.ui.platform.ComposeView
 import com.instructure.canvasapi2.models.CanvasContext
+import com.instructure.canvasapi2.utils.pageview.PageView
 import com.instructure.interactions.router.Route
 import com.instructure.pandautils.compose.CanvasTheme
 import com.instructure.pandautils.features.dashboard.notifications.DashboardRouter
@@ -32,6 +33,7 @@ import dagger.hilt.android.AndroidEntryPoint
 import javax.inject.Inject
 
 //This needed to be named NewDashboardFragment because Pendo tracking would break otherwise.
+@PageView
 @AndroidEntryPoint
 class NewDashboardFragment : ParentFragment() {
 
diff --git a/apps/student/src/main/java/com/instructure/student/features/modules/progression/StudioVideoFragment.kt b/apps/student/src/main/java/com/instructure/student/features/modules/progression/StudioVideoFragment.kt
@@ -24,6 +24,8 @@ import android.view.ViewGroup
 import androidx.compose.ui.graphics.Color
 import androidx.compose.ui.platform.ComposeView
 import com.instructure.canvasapi2.models.CanvasContext
+import com.instructure.canvasapi2.utils.pageview.PageView
+import com.instructure.canvasapi2.utils.pageview.PageViewUrl
 import com.instructure.interactions.router.Route
 import com.instructure.pandautils.compose.CanvasTheme
 import com.instructure.pandautils.utils.LongArg
@@ -39,6 +41,7 @@ private const val VIDEO_URI = "video_uri"
 private const val VIDEO_TITLE = "video_title"
 private const val POSTER_URI = "poster_uri"
 
+@PageView
 @AndroidEntryPoint
 class StudioVideoFragment : ParentFragment() {
 
@@ -47,6 +50,9 @@ class StudioVideoFragment : ParentFragment() {
     private var videoTitle: String by StringArg(key = VIDEO_TITLE)
     private var posterUri: String? by NullableStringArg(key = POSTER_URI)
 
+    @PageViewUrl
+    fun makePageViewUrl() = videoUri
+
     override fun title(): String = videoTitle
 
     override fun applyTheme() {}
diff --git a/apps/student/src/main/java/com/instructure/student/fragment/AssignmentBasicFragment.kt b/apps/student/src/main/java/com/instructure/student/fragment/AssignmentBasicFragment.kt
@@ -27,6 +27,9 @@ import com.instructure.canvasapi2.models.CanvasContext
 import com.instructure.canvasapi2.models.LockInfo
 import com.instructure.canvasapi2.utils.ApiPrefs
 import com.instructure.canvasapi2.utils.DateHelper
+import com.instructure.canvasapi2.utils.pageview.PageView
+import com.instructure.canvasapi2.utils.pageview.PageViewUrl
+import com.instructure.canvasapi2.utils.pageview.PageViewUrlParam
 import com.instructure.interactions.router.Route
 import com.instructure.pandautils.analytics.SCREEN_VIEW_ASSIGNMENT_BASIC
 import com.instructure.pandautils.analytics.ScreenView
@@ -58,6 +61,7 @@ import java.util.Locale
 import javax.inject.Inject
 
 @AndroidEntryPoint
+@PageView
 @ScreenView(SCREEN_VIEW_ASSIGNMENT_BASIC)
 class AssignmentBasicFragment : ParentFragment() {
 
@@ -70,6 +74,9 @@ class AssignmentBasicFragment : ParentFragment() {
     private var assignment: Assignment by ParcelableArg()
     private var loadHtmlJob: Job? = null
 
+    @PageViewUrl
+    fun makePageViewUrl() = assignment.htmlUrl.orEmpty()
+
     //region Fragment Lifecycle Overrides
     override fun onCreateView(inflater: LayoutInflater, container: ViewGroup?, savedInstanceState: Bundle?): View? =
             layoutInflater.inflate(R.layout.fragment_assignment_basic, container, false)
diff --git a/apps/student/src/main/java/com/instructure/student/mobius/elementary/ElementaryDashboardFragment.kt b/apps/student/src/main/java/com/instructure/student/mobius/elementary/ElementaryDashboardFragment.kt
@@ -24,6 +24,7 @@ import androidx.core.view.ViewCompat
 import androidx.core.view.WindowInsetsCompat
 import com.google.android.material.tabs.TabLayout
 import com.instructure.canvasapi2.models.CanvasContext
+import com.instructure.canvasapi2.utils.pageview.PageView
 import com.instructure.interactions.router.Route
 import com.instructure.pandautils.analytics.SCREEN_VIEW_ELEMENTARY_DASHBOARD
 import com.instructure.pandautils.analytics.ScreenView
@@ -42,6 +43,7 @@ import com.instructure.student.R
 import com.instructure.student.databinding.FragmentElementaryDashboardBinding
 import com.instructure.student.fragment.ParentFragment
 
+@PageView
 @ScreenView(SCREEN_VIEW_ELEMENTARY_DASHBOARD)
 class ElementaryDashboardFragment : ParentFragment() {
 
diff --git a/apps/teacher/src/main/java/com/instructure/teacher/adapters/StudentContextFragment.kt b/apps/teacher/src/main/java/com/instructure/teacher/adapters/StudentContextFragment.kt
@@ -35,6 +35,8 @@ import com.instructure.canvasapi2.type.EnrollmentType
 import com.instructure.canvasapi2.utils.DateHelper
 import com.instructure.canvasapi2.utils.Pronouns
 import com.instructure.canvasapi2.utils.isValid
+import com.instructure.canvasapi2.utils.pageview.PageView
+import com.instructure.canvasapi2.utils.pageview.PageViewUrlParam
 import com.instructure.interactions.MasterDetailInteractions
 import com.instructure.interactions.router.Route
 import com.instructure.interactions.router.RouteContext
@@ -80,6 +82,7 @@ import org.greenrobot.eventbus.Subscribe
 import org.greenrobot.eventbus.ThreadMode
 import javax.inject.Inject
 
+@PageView(url = "courses/{courseId}/users/{userId}")
 @AndroidEntryPoint
 class StudentContextFragment : PresenterFragment<StudentContextPresenter, StudentContextView>(), StudentContextView {
 
@@ -88,8 +91,11 @@ class StudentContextFragment : PresenterFragment<StudentContextPresenter, Studen
 
     private val binding by viewBinding(FragmentStudentContextBinding::bind)
 
-    private var mStudentId by LongArg()
-    private var mCourseId by LongArg()
+    @get:PageViewUrlParam("userId")
+    var mStudentId by LongArg()
+    @get:PageViewUrlParam("courseId")
+    var mCourseId by LongArg()
+
     private var mLaunchSubmissions by BooleanArg()
     private var needToForceNetwork = false
     private var mHasLoaded = false
diff --git a/apps/teacher/src/main/java/com/instructure/teacher/features/files/details/FileDetailsFragment.kt b/apps/teacher/src/main/java/com/instructure/teacher/features/files/details/FileDetailsFragment.kt
@@ -26,6 +26,8 @@ import androidx.fragment.app.Fragment
 import com.instructure.pandautils.base.BaseCanvasFragment
 import androidx.fragment.app.viewModels
 import com.instructure.canvasapi2.models.CanvasContext
+import com.instructure.canvasapi2.utils.pageview.PageView
+import com.instructure.canvasapi2.utils.pageview.PageViewUrl
 import com.instructure.canvasapi2.utils.tryOrNull
 import com.instructure.pandautils.binding.viewBinding
 import com.instructure.pandautils.utils.BooleanArg
@@ -43,12 +45,25 @@ import com.instructure.teacher.fragments.ViewUnsupportedFileFragment
 import dagger.hilt.android.AndroidEntryPoint
 
 @AndroidEntryPoint
+@PageView
 class FileDetailsFragment : BaseCanvasFragment() {
 
     private val viewModel: FileDetailsViewModel by viewModels()
     private val binding by viewBinding(FragmentFileDetailsBinding::bind)
     private var isInModulesPager: Boolean by BooleanArg(key = IS_IN_MODULES_PAGER, default = false)
 
+    @PageViewUrl
+    fun makePageViewUrl(): String {
+        val fileData = viewModel.data.value?.fileData ?: return ""
+        return when (fileData) {
+            is FileViewData.Pdf -> fileData.url
+            is FileViewData.Media -> fileData.url
+            is FileViewData.Image -> fileData.url
+            is FileViewData.Html -> fileData.url
+            is FileViewData.Other -> fileData.url
+        }
+    }
+
     override fun onCreateView(inflater: LayoutInflater, container: ViewGroup?, savedInstanceState: Bundle?): View {
         return inflater.inflate(R.layout.fragment_file_details, container, false)
     }
diff --git a/apps/teacher/src/main/java/com/instructure/teacher/features/modules/list/ui/ModuleListFragment.kt b/apps/teacher/src/main/java/com/instructure/teacher/features/modules/list/ui/ModuleListFragment.kt
@@ -24,6 +24,7 @@ import androidx.lifecycle.lifecycleScope
 import com.instructure.canvasapi2.apis.ModuleAPI
 import com.instructure.canvasapi2.apis.ProgressAPI
 import com.instructure.canvasapi2.models.CanvasContext
+import com.instructure.canvasapi2.utils.pageview.PageView
 import com.instructure.pandautils.features.progress.ProgressPreferences
 import com.instructure.pandautils.room.appdatabase.daos.ModuleBulkProgressDao
 import com.instructure.pandautils.utils.Const
@@ -33,6 +34,7 @@ import dagger.hilt.android.AndroidEntryPoint
 import kotlinx.coroutines.launch
 import javax.inject.Inject
 
+@PageView(url = "modules")
 @AndroidEntryPoint
 class ModuleListFragment : ModuleListMobiusFragment() {
 
diff --git a/apps/teacher/src/main/java/com/instructure/teacher/features/modules/progression/ModuleProgressionFragment.kt b/apps/teacher/src/main/java/com/instructure/teacher/features/modules/progression/ModuleProgressionFragment.kt
@@ -26,6 +26,8 @@ import androidx.fragment.app.viewModels
 import androidx.viewpager.widget.ViewPager.SimpleOnPageChangeListener
 import com.instructure.canvasapi2.models.CanvasContext
 import com.instructure.canvasapi2.models.Course
+import com.instructure.canvasapi2.utils.pageview.PageView
+import com.instructure.canvasapi2.utils.pageview.PageViewUrlParam
 import com.instructure.interactions.router.Route
 import com.instructure.interactions.router.RouterParams
 import com.instructure.pandautils.binding.viewBinding
@@ -48,13 +50,15 @@ import com.instructure.teacher.router.RouteMatcher
 import dagger.hilt.android.AndroidEntryPoint
 
 @AndroidEntryPoint
+@PageView("modules/{moduleItemId}")
 class ModuleProgressionFragment : BaseCanvasFragment() {
 
     private val viewModel: ModuleProgressionViewModel by viewModels()
     private val binding by viewBinding(FragmentModuleProgressionBinding::bind)
 
     private val canvasContext: CanvasContext by ParcelableArg(key = Const.CANVAS_CONTEXT)
-    private val moduleItemId by LongArg(key = RouterParams.MODULE_ITEM_ID, default = -1L)
+    @get:PageViewUrlParam("moduleItemId")
+    val moduleItemId by LongArg(key = RouterParams.MODULE_ITEM_ID, default = -1L)
     private val assetType by StringArg(key = ASSET_TYPE)
     private val assetId by StringArg(key = ASSET_ID)
     private val route: Route by ParcelableArg(key = ROUTE)
diff --git a/apps/teacher/src/main/java/com/instructure/teacher/features/syllabus/edit/EditSyllabusFragment.kt b/apps/teacher/src/main/java/com/instructure/teacher/features/syllabus/edit/EditSyllabusFragment.kt
@@ -22,6 +22,10 @@ import android.os.Bundle
 import android.view.LayoutInflater
 import android.view.ViewGroup
 import com.instructure.canvasapi2.models.Course
+import com.instructure.canvasapi2.utils.ApiPrefs
+import com.instructure.canvasapi2.utils.pageview.PageView
+import com.instructure.canvasapi2.utils.pageview.PageViewUrl
+import com.instructure.canvasapi2.utils.pageview.PageViewUrlParam
 import com.instructure.pandautils.analytics.SCREEN_VIEW_EDIT_SYLLABUS
 import com.instructure.pandautils.analytics.ScreenView
 import com.instructure.pandautils.interfaces.NavigationCallbacks
@@ -34,13 +38,17 @@ import com.instructure.teacher.mobius.common.ui.UpdateInit
 
 private const val SUMMARY_ALLOWED = "summaryAllowed"
 
+@PageView
 @ScreenView(SCREEN_VIEW_EDIT_SYLLABUS)
 class EditSyllabusFragment : MobiusFragment<EditSyllabusModel, EditSyllabusEvent, EditSyllabusEffect,
         EditSyllabusView, EditSyllabusViewState, FragmentEditSyllabusBinding>(), NavigationCallbacks {
 
     private val course by ParcelableArg<Course>()
     private val summaryAllowed: Boolean by BooleanArg(key = SUMMARY_ALLOWED)
 
+    @PageViewUrl
+    fun makePageViewUrl(): String = "${ApiPrefs.fullDomain}/courses/${course.id}/syllabus/edit"
+
     override fun makeEffectHandler(): EffectHandler<EditSyllabusView, EditSyllabusEvent, EditSyllabusEffect> = EditSyllabusEffectHandler()
 
     override fun makeUpdate(): UpdateInit<EditSyllabusModel, EditSyllabusEvent, EditSyllabusEffect> = EditSyllabusUpdate()
diff --git a/apps/teacher/src/main/java/com/instructure/teacher/features/syllabus/ui/SyllabusRepositoryFragment.kt b/apps/teacher/src/main/java/com/instructure/teacher/features/syllabus/ui/SyllabusRepositoryFragment.kt
@@ -20,12 +20,14 @@ package com.instructure.teacher.features.syllabus.ui
 import android.os.Bundle
 import com.instructure.canvasapi2.models.CanvasContext
 import com.instructure.canvasapi2.models.Course
+import com.instructure.canvasapi2.utils.pageview.PageView
 import com.instructure.pandautils.utils.Const
 import com.instructure.pandautils.utils.withArgs
 import com.instructure.teacher.features.syllabus.SyllabusRepository
 import dagger.hilt.android.AndroidEntryPoint
 import javax.inject.Inject
 
+@PageView(url = "assignments/syllabus")
 @AndroidEntryPoint
 class SyllabusRepositoryFragment : SyllabusFragment() {
 
diff --git a/libs/pandautils/src/main/java/com/instructure/pandautils/features/calendar/CalendarFragment.kt b/libs/pandautils/src/main/java/com/instructure/pandautils/features/calendar/CalendarFragment.kt
@@ -15,6 +15,7 @@
  */
 package com.instructure.pandautils.features.calendar
 
+import com.instructure.canvasapi2.utils.pageview.PageView
 import com.instructure.interactions.router.Route
 import com.instructure.pandautils.utils.withArgs
 import dagger.hilt.android.AndroidEntryPoint
diff --git a/libs/pandautils/src/main/java/com/instructure/pandautils/features/speedgrader/SpeedGraderFragment.kt b/libs/pandautils/src/main/java/com/instructure/pandautils/features/speedgrader/SpeedGraderFragment.kt
@@ -30,6 +30,8 @@ import com.instructure.canvasapi2.models.CanvasContext
 import com.instructure.canvasapi2.models.GradeableStudentSubmission
 import com.instructure.canvasapi2.models.StudentAssignee
 import com.instructure.canvasapi2.utils.coerceAtLeast
+import com.instructure.canvasapi2.utils.pageview.PageView
+import com.instructure.canvasapi2.utils.pageview.PageViewUrlParam
 import com.instructure.canvasapi2.utils.rangeWithin
 import com.instructure.interactions.router.Route
 import com.instructure.interactions.router.RouterParams
@@ -44,14 +46,18 @@ import dagger.hilt.android.AndroidEntryPoint
 import java.util.Locale
 
 @AndroidEntryPoint
+@PageView("courses/{courseId}/gradebook/speed_grader?assignment_id={assignmentId}")
 class SpeedGraderFragment : BaseCanvasFragment() {
 
+    @get:PageViewUrlParam("courseId")
+    val courseId: Long by LongArg(key = Const.COURSE_ID)
+    @get:PageViewUrlParam("assignmentId")
+    val assignmentId: Long by LongArg(key = Const.ASSIGNMENT_ID)
+
     private val viewModel: SpeedGraderViewModel by viewModels()
 
     private val sharedViewModel: SpeedGraderSharedViewModel by activityViewModels()
 
-    private val courseId by LongArg(key = Const.COURSE_ID)
-
     override fun onActivityCreated(savedInstanceState: Bundle?) {
         super.onActivityCreated(savedInstanceState)
         ThemePrefs.reapplyCanvasTheme(requireActivity())
