Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#2474 closed enhancement (fixed)

Forums: Add a name to pending and archived post notices

Reported by: sergeybiryukov's profile SergeyBiryukov Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: Priority: normal
Component: Support Forums Keywords: has-patch commit


In #2379, we've added reporter's username to "flagged as spam" notice:

This post has been flagged as spam by %s.

It would make sense to do the same for pending and archived posts:

  • Pending
    • If username is available: This post was unapproved by %s.
    • If not: This post is currently pending.
  • Archived
    • If username is available: This post was archived by %s.
    • If not: This post is currently archived..

At the moment, archived posts don't have any notice at all, which is inconsistent with spam and pending.

Attachments (1)

meta-2474.patch (3.8 KB) - added by SergeyBiryukov 7 years ago.

Download all attachments as: .zip

Change History (11)

#1 @coffee2code
7 years ago

In 4890:

Support Theme: Add moderation notice for posts that are archived.

See #2474.

#2 @SergeyBiryukov
7 years ago

In 4898:

Support Theme: After [4890], adjust the logic to only show moderation notice for archived posts to moderators.

Fixes #2489. See #2474.

This ticket was mentioned in Slack in #forums by sergey. View the logs.

7 years ago

#4 @SergeyBiryukov
7 years ago

  • Keywords has-patch commit added

meta-2474.patch stores the username of the moderator who performed the last approve/unapprove or archive/unarchive action, and displays that data in the notice, if available.

The name of the moderator who approved or unarchived the post is not displayed anywhere at the moment (as there's no notices for normal posts), but I thought it would be helpful to keep that data for future reference, just in case.

#5 @SergeyBiryukov
7 years ago

In 4987:

Support Forums: Store moderator's username on approve/unapprove and archive/unarchive actions.

See #2474.

#6 @SergeyBiryukov
7 years ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 4988:

Support Theme: Add moderator's username to the notice displayed for archived and pending posts.

Fixes #2474.

#7 @SergeyBiryukov
7 years ago

In 5111:

Support Forums: In Moderators::archive_post() and ::unarchive_post(), store user_nicename instead of user_login, for consistency with [5002].

See #2474, #2493.

#8 @SergeyBiryukov
7 years ago

In 5291:

Support Forums: Move Hooks::store_moderator_username() method added in [4987] to Moderators class.

See #2474.

#9 @SergeyBiryukov
7 years ago

In 5292:

Support Forums: In Moderators::store_moderator_username(), store user_nicename instead of user_login, for consistency with [5002] and [5111].

See #2474, #2493.

#10 @SergeyBiryukov
7 years ago

In 5293:

Support Forums: Use Moderators::store_moderator_username() in ::archive_post() and ::unarchive_post().

See #2474.

Note: See TracTickets for help on using tickets.