Skip to content

Add Notifications Mute status#149

Merged
sjp4 merged 1 commit intocoredevices:masterfrom
asyba:notification_mute_status
Apr 13, 2026
Merged

Add Notifications Mute status#149
sjp4 merged 1 commit intocoredevices:masterfrom
asyba:notification_mute_status

Conversation

@asyba
Copy link
Copy Markdown
Contributor

@asyba asyba commented Apr 6, 2026

  • A bug was fixed where the mute toggle was not reflecting the correct status when the muteExpiration setting had an expired date and was subsequently reactivated.

  • A status was added to the notification app view that indicates whether the mute is always active, active on weekends/weekdays, or only active on specific hours/days.

IMG_1999 IMG_1992 IMG_1994

}
}
} else {
androidx.compose.foundation.layout.Spacer(modifier = Modifier.weight(1f))
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be able to import this

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

}

val expiration = app.muteExpiration
val now = kotlin.time.Clock.System.now()
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

import

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

HorizontalDivider(modifier = Modifier.padding(top = 8.dp))
}
}
} else {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nesting here isn't very clear - might be better to have a separate if (iOS) { } block

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@asyba asyba force-pushed the notification_mute_status branch from c847986 to 5542828 Compare April 9, 2026 00:45
@asyba asyba requested a review from sjp4 April 9, 2026 00:46
@sjp4 sjp4 merged commit 66fbf3c into coredevices:master Apr 13, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants