fix: Fixes according to PR feedback

Author: PavloNetrebchuk

core/src/main/java/org/openedx/core/data/model/CourseAssignments.kt

@@ -2,22 +2,22 @@ package org.openedx.core.data.model
 
 import com.google.gson.annotations.SerializedName
 import org.openedx.core.data.model.room.discovery.CourseAssignmentsDb
+import org.openedx.core.domain.model.CourseAssignments
 
 data class CourseAssignments(
     @SerializedName("future_assignments")
     val futureAssignments: List<CourseDateBlock>?,
     @SerializedName("past_assignments")
     val pastAssignments: List<CourseDateBlock>?
 ) {
-    fun mapToDomain(): org.openedx.core.domain.model.CourseAssignments =
-        org.openedx.core.domain.model.CourseAssignments(
-            futureAssignments = futureAssignments?.map {
-                it.mapToDomain()
-            },
-            pastAssignments = pastAssignments?.map {
-                it.mapToDomain()
-            }
-        )
+    fun mapToDomain() = CourseAssignments(
+        futureAssignments = futureAssignments?.map {
+            it.mapToDomain()
+        },
+        pastAssignments = pastAssignments?.map {
+            it.mapToDomain()
+        }
+    )
 
     fun mapToRoomEntity() = CourseAssignmentsDb(
         futureAssignments = futureAssignments?.map {

core/src/main/java/org/openedx/core/data/model/CourseDateBlock.kt

@@ -4,6 +4,7 @@ import android.os.Parcelable
 import com.google.gson.annotations.SerializedName
 import kotlinx.parcelize.Parcelize
 import org.openedx.core.data.model.room.discovery.CourseDateBlockDb
+import org.openedx.core.domain.model.CourseDateBlock
 import org.openedx.core.utils.TimeUtils
 import java.util.Date
 
@@ -31,7 +32,7 @@ data class CourseDateBlock(
     @SerializedName("first_component_block_id")
     val blockId: String = "",
 ): Parcelable {
-    fun mapToDomain() = org.openedx.core.domain.model.CourseDateBlock(
+    fun mapToDomain() = CourseDateBlock(
         complete = complete,
         date = TimeUtils.iso8601ToDate(date) ?: Date(),
         assignmentType = assignmentType,

core/src/main/java/org/openedx/core/data/model/CourseStatus.kt

@@ -2,6 +2,7 @@ package org.openedx.core.data.model
 
 import com.google.gson.annotations.SerializedName
 import org.openedx.core.data.model.room.discovery.CourseStatusDb
+import org.openedx.core.domain.model.CourseStatus
 
 data class CourseStatus(
     @SerializedName("last_visited_module_id")
@@ -13,13 +14,12 @@ data class CourseStatus(
     @SerializedName("last_visited_unit_display_name")
     val lastVisitedUnitDisplayName: String?
 ) {
-    fun mapToDomain(): org.openedx.core.domain.model.CourseStatus =
-        org.openedx.core.domain.model.CourseStatus(
-            lastVisitedModuleId = lastVisitedModuleId ?: "",
-            lastVisitedModulePath = lastVisitedModulePath ?: emptyList(),
-            lastVisitedBlockId = lastVisitedBlockId ?: "",
-            lastVisitedUnitDisplayName = lastVisitedUnitDisplayName ?: ""
-        )
+    fun mapToDomain() = CourseStatus(
+        lastVisitedModuleId = lastVisitedModuleId ?: "",
+        lastVisitedModulePath = lastVisitedModulePath ?: emptyList(),
+        lastVisitedBlockId = lastVisitedBlockId ?: "",
+        lastVisitedUnitDisplayName = lastVisitedUnitDisplayName ?: ""
+    )
 
     fun mapToRoomEntity() = CourseStatusDb(
         lastVisitedModuleId = lastVisitedModuleId ?: "",

core/src/main/java/org/openedx/core/data/model/EnrolledCourse.kt

@@ -5,6 +5,7 @@ import org.openedx.core.data.model.room.discovery.EnrolledCourseEntity
 import org.openedx.core.data.model.room.discovery.ProgressDb
 import org.openedx.core.domain.model.EnrolledCourse
 import org.openedx.core.utils.TimeUtils
+import org.openedx.core.domain.model.Progress as ProgressDomain
 
 data class EnrolledCourse(
     @SerializedName("audit_access_expires")
@@ -34,7 +35,7 @@ data class EnrolledCourse(
             isActive = isActive ?: false,
             course = course?.mapToDomain()!!,
             certificate = certificate?.mapToDomain(),
-            progress = progress?.mapToDomain() ?: org.openedx.core.domain.model.Progress.DEFAULT_PROGRESS,
+            progress = progress?.mapToDomain() ?: ProgressDomain.DEFAULT_PROGRESS,
             courseStatus = courseStatus?.mapToDomain(),
             courseAssignments = courseAssignments?.mapToDomain()
         )

core/src/main/java/org/openedx/core/data/model/Progress.kt

@@ -2,19 +2,18 @@ package org.openedx.core.data.model
 
 import com.google.gson.annotations.SerializedName
 import org.openedx.core.data.model.room.discovery.ProgressDb
+import org.openedx.core.domain.model.Progress
 
 data class Progress(
     @SerializedName("assignments_completed")
     val assignmentsCompleted: Int?,
     @SerializedName("total_assignments_count")
     val totalAssignmentsCount: Int?
 ) {
-    fun mapToDomain(): org.openedx.core.domain.model.Progress {
-        return org.openedx.core.domain.model.Progress(
-            assignmentsCompleted = assignmentsCompleted ?: 0,
-            totalAssignmentsCount = totalAssignmentsCount ?: 0
-        )
-    }
+    fun mapToDomain() = Progress(
+        assignmentsCompleted = assignmentsCompleted ?: 0,
+        totalAssignmentsCount = totalAssignmentsCount ?: 0
+    )
 
     fun mapToRoomEntity() = ProgressDb(
         assignmentsCompleted = assignmentsCompleted ?: 0,

core/src/main/java/org/openedx/core/presentation/global/InDevelopmentScreen.kt

@@ -28,4 +28,4 @@ fun InDevelopmentScreen(
             style = MaterialTheme.appTypography.headlineMedium
         )
     }
-}
+}

core/src/main/java/org/openedx/core/ui/theme/Type.kt

@@ -17,7 +17,7 @@ data class AppTypography(
     val displayLarge: TextStyle,
     val displayMedium: TextStyle,
     val displaySmall: TextStyle,
-    val headlineBolt: TextStyle,
+    val headlineBold: TextStyle,
     val headlineLarge: TextStyle,
     val headlineMedium: TextStyle,
     val headlineSmall: TextStyle,
@@ -74,7 +74,7 @@ internal val LocalTypography = staticCompositionLocalOf {
             letterSpacing = 0.sp,
             fontFamily = fontFamily
         ),
-        headlineBolt = TextStyle(
+        headlineBold = TextStyle(
             fontSize = 34.sp,
             lineHeight = 24.sp,
             fontWeight = FontWeight.Bold,

core/src/main/java/org/openedx/core/utils/FileUtil.kt

@@ -31,8 +31,6 @@ class FileUtil(val context: Context) {
             null
         }
     }
-
-
 }
 
 enum class Directories {

dashboard/src/main/java/org/openedx/DashboardUI.kt

@@ -46,4 +46,4 @@ private fun LockPreview() {
     OpenEdXTheme {
         Lock()
     }
-}
+}

dashboard/src/main/java/org/openedx/courses/presentation/AllEnrolledCoursesFragment.kt

@@ -530,7 +530,7 @@ private fun Header(
             modifier = Modifier.align(Alignment.CenterStart),
             text = stringResource(id = R.string.dashboard_all_courses),
             color = MaterialTheme.appColors.textDark,
-            style = MaterialTheme.appTypography.headlineBolt
+            style = MaterialTheme.appTypography.headlineBold
         )
         IconButton(
             modifier = Modifier