Skip to content

Conversation

@pfefferle
Copy link
Member

Fixes #1421 (comment)

Proposed changes:

  • The Undo event now deletes reactions instead of trashing them.

Other information:

  • Have you written new tests for your changes, if applicable?

Testing instructions:

  • Like or Boost a WordPress post on Mastodon et. al.
  • Undo this action
  • Check the trash

Changelog entry

  • Automatically create a changelog entry from the details below.
Changelog Entry Details

Significance

  • Patch
  • Minor
  • Major

Type

  • Added - for new features
  • Changed - for changes in existing functionality
  • Deprecated - for soon-to-be removed features
  • Removed - for now removed features
  • Fixed - for any bug fixes
  • Security - in case of vulnerabilities

Message

Permanently delete reactions that were Undo instead of trashing them.

@pfefferle pfefferle self-assigned this Mar 28, 2025
@pfefferle pfefferle added the [Type] Bug Something isn't working label Mar 28, 2025
@pfefferle pfefferle requested review from a team and jeherve March 28, 2025 11:26
@pfefferle pfefferle merged commit 7e1c9e1 into trunk Mar 28, 2025
13 checks passed
@pfefferle pfefferle deleted the fix/permanent-delete branch March 28, 2025 12:43
jsit added a commit to jsit/wordpress-activitypub that referenced this pull request Nov 1, 2025
* tag '5.6.1': (47 commits)
  Release 5.6.1 (Automattic#1542)
  Fix: Use specified time formatting for Outbox Activities (Automattic#1537)
  Fix: Default value for empty `movedTo` field (Automattic#1539)
  Fix: Post Interactions settings (Automattic#1540)
  Release 5.6.0 (Automattic#1534)
  lover significance (Automattic#1533)
  Scheduler: Don't federate Deletes of unfederated posts. (Automattic#1528)
  Move: use `$from` for object in `Move::externally` (Automattic#1531)
  Fix: Properly traverse the object hierarchy to get the ID (Automattic#1518)
  Mentions: Standardize around only displaying username (Automattic#1510)
  Tests: Reduce chance of time offsets (Automattic#1527)
  Import: Don't federate new attachments (Automattic#1526)
  Importer: Format import count number. (Automattic#1525)
  Move: Add a Mastodon importer (Automattic#1502)
  Move: use `$from` for object in `Move::internally` (Automattic#1516)
  Fix: Do not send self-replies (Automattic#1517)
  Fix: Delete Undone comments instead of trashing them (Automattic#1520)
  Dispatcher: Adhere to passed batch size. (Automattic#1514)
  Add interaction settings (Automattic#1395)
  Fix: Set proper default, to show Welcome screen by default (Automattic#1512)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Type] Bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fediverse Reactions

4 participants