Skip to content

Fix user unable to re-like a comment - #13282

Merged
dylanjeffers merged 1 commit into
mainfrom
fix-relike-bug
Oct 21, 2025
Merged

Fix user unable to re-like a comment#13282
dylanjeffers merged 1 commit into
mainfrom
fix-relike-bug

Conversation

@dylanjeffers

Copy link
Copy Markdown
Contributor

Description

Fixes issue where indexer incorrectly thinks a user has liked a comment due to failure to check if reaction is deleted or not

@changeset-bot

changeset-bot Bot commented Oct 21, 2025

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: e3d4f81

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

@raymondjacobson

Copy link
Copy Markdown
Member

❤️

@dylanjeffers
dylanjeffers merged commit f16fda1 into main Oct 21, 2025
4 of 6 checks passed
@dylanjeffers
dylanjeffers deleted the fix-relike-bug branch October 21, 2025 17:45
audius-infra pushed a commit that referenced this pull request Oct 22, 2025
[a6d5e9d] Use production wAUDIO on stage (#13288) Marcus Pasell
[f16fda1] Fix user unable to re-like a comment (#13282) Dylan Jeffers
rickyrombo added a commit that referenced this pull request Oct 28, 2025
rickyrombo added a commit that referenced this pull request Oct 28, 2025
Indexing was getting stalled from the previous fix because the primary
key would already exist.

This change reverts #13282 and fixes it a different way by moving the
check for existing deleted comments to the validation logic, and adding
the code to handle that case in the reaction creation logic.

Also adds commands for `audius-cmd` and tested locally.

**NOTE**: Re-liking a comment will _not_ trigger a new notification.
Made a call that we probably don't want to spam notifs every time it
toggles.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants