Skip to content

Commit

Permalink
Refactor: Remove unused NotificationScreenModel from ReminderScreen
Browse files Browse the repository at this point in the history
Removed the unused `NotificationScreenModel` parameter from the `ReminderSheet` composable in `Reminder
Screen`. This change removes unnecessary dependencies and simplifies the code.
  • Loading branch information
youndon committed Nov 17, 2024
1 parent 92a8843 commit bf5a900
Showing 1 changed file with 0 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@ import cafe.adriel.voyager.navigator.Navigator
import cafe.adriel.voyager.navigator.bottomSheet.LocalBottomSheetNavigator
import city.zouitel.logic.events.UiEvent
import city.zouitel.notifications.viewmodel.AlarmManagerScreenModel
import city.zouitel.notifications.viewmodel.NotificationScreenModel
import city.zouitel.reminder.layout.DateLayout
import city.zouitel.reminder.layout.TimeLayout
import city.zouitel.reminder.model.Reminder
Expand All @@ -47,13 +46,11 @@ data class ReminderScreen(
val uid: String,
val title: String?,
val message: String?,
val remindingValue: (Long) -> Unit
): Screen {
@Composable
override fun Content() {
ReminderSheet(
dataStoreModel = getScreenModel(),
notificationModel = getScreenModel(),
reminderModel = getScreenModel(),
alarmManagerScreenModel = getScreenModel()
)
Expand All @@ -63,7 +60,6 @@ data class ReminderScreen(
@Composable
private fun ReminderSheet(
dataStoreModel: DataStoreScreenModel,
notificationModel: NotificationScreenModel,
reminderModel: ReminderScreenModel,
alarmManagerScreenModel: AlarmManagerScreenModel
) {
Expand Down

0 comments on commit bf5a900

Please sign in to comment.