Deleting post in moderated group queue does not remove post
Summary
If a user deletes a post in a moderated group before it is approved, the post can still be approved and does not get deleted.
Steps to reproduce
- Make a group, make it moderated.
- Login to a secondary account, create posts numbered 1, 2, 3, 4, 5.
- Log-in to primary account, go to queue and reject the post numbered 3.
- Observe the post is no longer in the queue, but if you go back to the group, it is still there with "this post is awaiting moderation" text.
Platform information
Cross-platform web
Relevant logs and/or screenshots
Designs
- Show closed items
Activity
-
Newest first Oldest first
-
Show all activity Show comments only Show history only
- Nicholas Lewis added Priority1 - High ProductGroups TypeBug (Triage) labels
added Priority1 - High ProductGroups TypeBug (Triage) labels
- Mark Harding assigned to @eiennohi
assigned to @eiennohi
- Mark Harding changed weight to 2
changed weight to 2
- Mark Harding changed milestone to %sprint: Hipster Hedgehog
changed milestone to %sprint: Hipster Hedgehog
- Marcelo Rivera added StatusInProgress label
added StatusInProgress label
- Marcelo Rivera mentioned in merge request !222 (closed)
mentioned in merge request !222 (closed)
- Marcelo Rivera added StatusReview label and removed StatusInProgress label
added StatusReview label and removed StatusInProgress label
- Marcelo Rivera mentioned in merge request !241 (closed)
mentioned in merge request !241 (closed)
- Mark Harding removed StatusReview label
removed StatusReview label
- Mark Harding added 1 deleted label
added 1 deleted label
- Ben added T - Triage TypeBug labels and removed TypeBug (Triage) label
added T - Triage TypeBug labels and removed TypeBug (Triage) label
- Ben removed T - Triage label
removed T - Triage label
- Mark Harding removed milestone
removed milestone
- Mark Harding changed title from (bug): deleting post in moderated group queue does not remove post to Deleting post in moderated group queue does not remove post
changed title from (bug): deleting post in moderated group queue does not remove post to Deleting post in moderated group queue does not remove post
- Mark Harding assigned to @benhayward.ben
assigned to @benhayward.ben
- Mark Harding changed time estimate to 2h
changed time estimate to 2h
- Mark Harding added StatusBacklog + 1 deleted label
added StatusBacklog + 1 deleted label
- Mark Harding unassigned @eiennohi
unassigned @eiennohi
- Ben added StatusInProgress label and removed StatusBacklog label
added StatusInProgress label and removed StatusBacklog label
- Reporter
By my testing this is not happening anymore. @javanick can you confirm?
- Posted in a moderated group.
- Deleted the post.
- Switched to moderator account.
- Post not there to approve.
- Ben added 30m of time spent at 2019-10-09
added 30m of time spent at 2019-10-09
- Ben added StatusFollow Up label and removed StatusInProgress label
added StatusFollow Up label and removed StatusInProgress label
- Nicholas Lewis added Priority2 - Medium label and removed Priority1 - High label
added Priority2 - Medium label and removed Priority1 - High label
- Ben removed StatusFollow Up label
removed StatusFollow Up label
- Ben added StatusFollow Up label
added StatusFollow Up label
- Ben removed StatusFollow Up label
removed StatusFollow Up label
- Ben added StatusInProgress label
added StatusInProgress label
- Ben changed the description
changed the description
- Reporter
Nick clarified the steps for me, can now reproduce.
- Ben moved to benhayward.ben/front#3 (closed)
moved to benhayward.ben/front#3 (closed)
- Ben reopened
reopened
- Ben mentioned in merge request !374 (merged)
mentioned in merge request !374 (merged)
- Ben added StatusReview label and removed StatusInProgress label
added StatusReview label and removed StatusInProgress label
- Ben added 2h of time spent at 2019-10-16
added 2h of time spent at 2019-10-16
- Reporter
Passed this deletion off to the event dispatcher
- Ben added 2h of time spent at 2019-11-19
added 2h of time spent at 2019-11-19
- Ben added 1 deleted label and removed StatusReview label
added 1 deleted label and removed StatusReview label
- Ben added StatusRequires Changes label and removed 1 deleted label
added StatusRequires Changes label and removed 1 deleted label
- Ben added 1 deleted label and removed 1 deleted label
added 1 deleted label and removed 1 deleted label
- Ben added 1 deleted label and removed StatusRequires Changes label
added 1 deleted label and removed StatusRequires Changes label
- Ben added StatusRequires Changes label and removed 1 deleted label
added StatusRequires Changes label and removed 1 deleted label
- Ben added 1 deleted label and removed StatusRequires Changes label
added 1 deleted label and removed StatusRequires Changes label
- Ben added StatusBlocked label and removed 1 deleted label
added StatusBlocked label and removed 1 deleted label
- Ben added StatusInProgress label and removed StatusBlocked label
added StatusInProgress label and removed StatusBlocked label
- Ben added StatusBlocked label and removed StatusInProgress label
added StatusBlocked label and removed StatusInProgress label
- Brian Hatchet added StatusFollow Up label and removed StatusBlocked label
added StatusFollow Up label and removed StatusBlocked label
- Brian Hatchet added 1 deleted label and removed StatusFollow Up label
added 1 deleted label and removed StatusFollow Up label
- Brian Hatchet added StatusInProgress label and removed 1 deleted label
added StatusInProgress label and removed 1 deleted label
- Contributor
Review site had a bad configuration. It needed to be completely stopped and restarted.
- Ben added StatusRequires Changes label and removed StatusInProgress label
added StatusRequires Changes label and removed StatusInProgress label
- Ben added 1 deleted label and removed StatusRequires Changes label
added 1 deleted label and removed StatusRequires Changes label
- Emiliano Balbuena assigned to @markeharding and @brianhatchet and unassigned @edgebal
assigned to @markeharding and @brianhatchet and unassigned @edgebal
- Emiliano Balbuena added StatusReview label and removed 1 deleted label
added StatusReview label and removed 1 deleted label
- Brian Hatchet added 1 deleted label and removed StatusReview label
added 1 deleted label and removed StatusReview label
- Xander Miller closed via commit c8ecd135
closed via commit c8ecd135
- Xander Miller added 1 deleted label and removed 1 deleted label
added 1 deleted label and removed 1 deleted label
- Xander Miller mentioned in commit c8ecd135
mentioned in commit c8ecd135