Skip to content
Snippets Groups Projects

some fixes for note edits

Merged dakkar requested to merge dakkar/Sharkey-thenautilus:fix/edit-replies into develop

What does this PR do?

we got reports of people editing replies and seeing them unlinked from the original post

I can't replicate the problem, but as @kopper said on Discord, there's no real reason to even change the replyId of a note!

So I "just" changed the code to always preserve it, regardless of what the frontend says.

While I was there, I also fixed a problem with the note_unread repository, where repeatedly editing a note could cause a "duplicate key" error from the database.

Contribution Guidelines By submitting this merge request, you agree to follow our Contribution Guidelines

  • I agree to follow this project's Contribution Guidelines
  • I have made sure to test this pull request

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading