Skip to content

[SDESK-7942]: Item sent to Personal space still available in the Sent Desk Output stage#5192

Merged
BrianMwangi21 merged 2 commits into
superdesk:release/2.10from
BrianMwangi21:SDESK-7942
May 29, 2026
Merged

[SDESK-7942]: Item sent to Personal space still available in the Sent Desk Output stage#5192
BrianMwangi21 merged 2 commits into
superdesk:release/2.10from
BrianMwangi21:SDESK-7942

Conversation

@BrianMwangi21

@BrianMwangi21 BrianMwangi21 commented May 28, 2026

Copy link
Copy Markdown
Contributor

Purpose

This PR fixes an issue where items copied to Personal Space were still appearing in the originating desk’s Sent Desk Output stage. This caused the same item to be visible in both places and made edits in Personal Space appear in Sent Desk Output.

What has changed

  • Updated sentDeskOutput monitoring filtering to exclude Personal Space items.
  • Added test

Steps to test

  1. Enable Sent Desk Output on a desk.
  2. Create an article on Desk 1 and send it to Desk 2.
  3. Confirm it appears in Desk 1 Sent Desk Output.
  4. From Desk 1, send that item to Personal Space.
  5. Refresh Desk 1 Sent Desk Output and confirm the Personal Space copy is no longer visible.
  6. Confirm the item still appears in Personal Space and can be edited there without leaking back into Sent Desk Output.

Checklist

  • I reviewed my code
  • I tested all affected functionality and made sure it works

Resolves: SDESK-7942

@BrianMwangi21 BrianMwangi21 requested a review from petrjasek May 29, 2026 06:57
Comment thread scripts/apps/monitoring/services/CardsService.ts Outdated
@BrianMwangi21 BrianMwangi21 merged commit 824f53f into superdesk:release/2.10 May 29, 2026
6 checks passed
BrianMwangi21 added a commit that referenced this pull request Jun 1, 2026
… Desk Output stage (#5192) (#5198)

* SDESK-7942: hide personal items from sent desk output

* suggested fixes
BrianMwangi21 added a commit that referenced this pull request Jun 5, 2026
… Desk Output stage (#5192) (#5210)

* SDESK-7942: hide personal items from sent desk output

* suggested fixes
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.

2 participants