delete note on tag UI

[vicky7230]

PR Type

Enhancement

---

Description

• Added UpdateNoteDeletedUseCase to TagsViewModelTest

• Introduced onNoteDeleteClick in TagScreenUi

• Added markNoteAsDeleted test case

---

Changes diagram

flowchart LR
  A["Add UpdateNoteDeletedUseCase"] -- "to TagsViewModelTest" --> B["Introduce onNoteDeleteClick"]
  B -- "in TagScreenUi" --> C["Add markNoteAsDeleted test"]

Loading

---

Changes walkthrough 📝

	Relevant files
Enhancement	2 files TagsViewModelTest.kt Added `UpdateNoteDeletedUseCase` and test case                      +59/-11  TagScreenUi.kt Introduced `onNoteDeleteClick` parameter                                  +3/-0
Additional files	8 files ChatHistory_schema_v2.xml +3/-0      androidTestResultsUserPreferences.xml +1/-0      deploymentTargetSelector.xml +11/-0    .pr_agent.toml +1/-1      Note.kt +1/-0      InternalTagsFeatureApi.kt +1/-0      TagsViewModel.kt +13/-4    TagContent.kt +34/-7

---

Need help?

Type /help how to ... in the comments thread for any questions about PR-Agent usage.

Check out the documentation for more information.

[github-actions]

Failed to generate code suggestions for PR

[github-actions]

Failed to generate code suggestions for PR

[github-actions]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 4 🔵🔵🔵🔵⚪

🧪 PR contains tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Possible Issue The SwipeableItemWithActions component is used to wrap TagNoteItem, which might lead to unexpected behavior if the swipe action is not properly handled or if the TagNoteItem does not support being wrapped in this way. SwipeableItemWithActions( modifier = Modifier.animateItem(), isRevealed = note.optionRevealed, actions = { ActionIcon( onClick = { onNoteDeleteClick(note) }, backgroundColor = Red, icon = ImageVector.vectorResource(id = com.core.common.R.drawable.ic_delete), modifier = Modifier.fillMaxHeight(), contentDescription = "Delete Icon", ) }, ) { TagNoteItem( modifier = Modifier.fillMaxWidth().background(tagsUiState.tag.color.toColorSafely()), note = note, onNoteClick = onNoteClick, onNoteDoneClick = onNoteDoneClick, ) } Code Duplication The onNoteDeleteClick handler is directly passed to TagContent. Ensure that this handler is not duplicating functionality or causing redundancy in the TagsViewModel. onNoteDeleteClick = tagsViewModel::markNoteAsDeleted,

[github-actions]

PR Code Suggestions ✨