From 475793059b1c1c0c141b4c65129db083759eb526 Mon Sep 17 00:00:00 2001 From: austinried <4966622+austinried@users.noreply.github.com> Date: Sun, 27 Sep 2026 12:50:38 +0000 Subject: [PATCH] keep main's setStar when resolving the rebase The conflict resolution took my side of ItemActions.setStar, which still suspended and returned Result; main's rework made it a plain action the menu calls without a coroutine, with the star filled optimistically. Compiles again, and ItemContextMenuStarTest passes. --- .../main/kotlin/com/subtracks/ui/components/ItemContextMenu.kt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/src/main/kotlin/com/subtracks/ui/components/ItemContextMenu.kt b/app/src/main/kotlin/com/subtracks/ui/components/ItemContextMenu.kt index e8eefb07..61268f07 100644 --- a/app/src/main/kotlin/com/subtracks/ui/components/ItemContextMenu.kt +++ b/app/src/main/kotlin/com/subtracks/ui/components/ItemContextMenu.kt @@ -131,7 +131,7 @@ class ItemActions( val downloadArtist: ((ArtistModel) -> Unit)? = null, val cancelArtistDownload: ((ArtistModel) -> Unit)? = null, val deleteArtistDownload: ((ArtistModel) -> Unit)? = null, - val setStar: suspend (StarType, String, Boolean) -> Result = { _, _, _ -> Result.success(Unit) }, + val setStar: (StarType, String, Boolean) -> Unit = { _, _, _ -> }, val viewAlbum: ((String) -> Unit)? = null, val viewArtist: ((String) -> Unit)? = null, )