Skip to content

Feature/ba 3286 rn delete a comment#381

Merged
nossila merged 1 commit into
epic/BA-2239-react-native-commentsfrom
feature/BA-3286-rn-delete-a-comment
Jun 18, 2026
Merged

Feature/ba 3286 rn delete a comment#381
nossila merged 1 commit into
epic/BA-2239-react-native-commentsfrom
feature/BA-3286-rn-delete-a-comment

Conversation

@davidbermudez-tsl

Copy link
Copy Markdown
Collaborator

As a user who owns a comment or has permission to moderate,
I want to be able to delete a comment in the mobile app,
So that I can manage and clean up content I no longer want to be visible, just like I can on the web.

Description:
This feature enables users to delete their own comments (or comments they have permission to moderate) directly from the mobile app. The delete action should be accessible through an overlay drawer.We already have this functionality on web.

Scenario: Display delete option for a comment in the app

Given the user is viewing a comment in the mobile app, and holds press over it, the overlay drawer displays the message "Delete Comment" in red.

Scenario: Show delete confirmation dialog before deleting comment

Given the user taps the delete action for a comment, when the system triggers the deletion flow, then a confirmation dialog appears with the message “Are you sure you want to delete this comment? This action cannot be undone.”

And the dialog includes two buttons: “Cancel” and “Delete”

And the “Delete” button is styled as a destructive action in red

Scenario: Cancel deletion from confirmation dialog

Given the confirmation dialog is visible, when the user taps “Cancel” or closes the dialog, then the system dismisses the dialog and no changes are made

Scenario: Confirm deletion from confirmation dialog

Given the confirmation dialog is visible, when the user taps “Delete”, then the system deletes the comment and removes it from the thread

And the comment is no longer visible in the comment list

Scenario: Ensure parity with web behavior

Given the delete feature works on web, when the same user attempts to delete a comment in the app, then the app provides the same functionality and confirmation flow as the web version

https://www.figma.com/design/z5ZdVlKEMmGIX4GVnjwfxA/BaseApp---NATIVE?node-id=9612-33758&t=PQHECmyjd8eFKcVb-1

https://www.figma.com/design/z5ZdVlKEMmGIX4GVnjwfxA/BaseApp---NATIVE?node-id=9612-33049&t=PQHECmyjd8eFKcVb-1

Approvd
https://app.approvd.io/silverlogic/BA/stories/44378

@changeset-bot

changeset-bot Bot commented Jun 5, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 6e78157

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@coderabbitai

coderabbitai Bot commented Jun 5, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 3123a5df-7120-488b-97cb-9b07896f079c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/BA-3286-rn-delete-a-comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@anicioalexandre anicioalexandre left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Approving the merge to the epic, but we still need to do a better review on the comments epic and fix a few things

@nossila nossila force-pushed the epic/BA-2239-react-native-comments branch from 37b9187 to a73e110 Compare June 18, 2026 15:15
@nossila nossila requested a review from matheusysd as a code owner June 18, 2026 15:15
@nossila nossila force-pushed the feature/BA-3286-rn-delete-a-comment branch from 3abb942 to 6e78157 Compare June 18, 2026 16:53
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
19.3% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@nossila nossila merged commit e04d886 into epic/BA-2239-react-native-comments Jun 18, 2026
6 of 7 checks passed
@nossila nossila deleted the feature/BA-3286-rn-delete-a-comment branch June 18, 2026 16:58
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.

4 participants