Skip to content

Freescout: Plugins Team additions. - #3

Merged
obenland merged 13 commits into
obenland:add/freescout-plugin-reviewfrom
frantorres:fran/freescout-fixes
Oct 16, 2026
Merged

obenland merged 13 commits into
obenland:add/freescout-plugin-reviewfrom
frantorres:fran/freescout-fixes

Conversation

@frantorres

@frantorres frantorres commented Oct 13, 2026 •

Copy link
Copy Markdown

Changes over the Plugin Review module to adapt it to the plugins team's workflows.

Plugin Review module

  • My Reviews and My Cases folders in the plugins mailbox: conversations the agent is participating in, is mentioned in, and that need a reviewer or volunteer. Includes bulk actions, switching to another member, and a tag filter for My Cases.
  • Buttons that open a section's conversations in new tabs in bulk, oldest first or by review progress, with a per-agent limit in the profile.
  • Copy X URLs in the header: copies the download URLs of the plugins in review the agent has open, in the order they were opened, for the local review tool.
  • Plugin directory subjects shortened to a mark and the plugin name, e.g. 🔌🛡️ My Plugin. The panel uses that subject as its title, with some visual changes.

Plugins & Themes sidebar panel

  • A status line under each name: an icon, the status, and the close reason or reviewer. It replaces the badges.
  • The last line has the slug, the last update as a short age (1d, 2mo), and the links aligned on the right: Gandalf (released plugins and published themes), the WordPress.org page, and the download.
  • A +/− after each plugin's download link adds or removes it from Copy X URLs.
  • Plugins in review are listed first and rejected ones last. The local mock panel has a sample plugin in each status.
  • The Gandalf URL and the "released" check are now shared between the panel and the Plugin Review endpoint.

Important for migration

  • \Modules\WPOrgPluginReview\Services\Reviewers::REVIEW_TEAM has to match the "Plugin Review" team name.
  • \Modules\WPOrgPluginReview\Services\Reviewers::ADMIN_TEAM has to match the "Plugin Administration" team name (it's currently different in HelpScout and FreeScout).

@github-actions

github-actions Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

Core Committers: Use this line as a base for the props when committing in SVN:

Props frantorres, obenland.

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@frantorres
frantorres marked this pull request as draft October 13, 2026 18:09
@frantorres
frantorres marked this pull request as ready for review October 16, 2026 17:15
@obenland
obenland force-pushed the add/freescout-plugin-review branch from 29fffbc to 71c8135 Compare October 16, 2026 19:57
@obenland
obenland force-pushed the fran/freescout-fixes branch from cec5c1b to c8feb89 Compare October 16, 2026 20:05
frantorres and others added 13 commits October 16, 2026 16:26
…relative time.

"Updated 1 day ago" becomes "(1d)", with the full date in the tooltip, to fit the narrow sidebar.
The local environment's recorded panel uses the same format.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…nd aligned links.

Each plugin or theme now shows its name across the full width, then, unless it's published, a status line with an icon
and the status in words (the close reason, and the reviewer while in review), instead of a badge; delisted themes get
no icon. The slug line ends with the last update and the links: Gandalf, for released plugins and published themes, the
page on WordPress.org, once approved, and the download, which most have, at the right edge.

The Gandalf URL and what makes a plugin released move to common.php, shared with the Plugin Review panel.

Lists show those in review first and rejected ones last. The local mock panel uses the new format, with a sample
plugin in each review status.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…d URL from the ones to copy.

A button after each plugin's download link adds its URL to the header's Copy X URLs, or removes it once it's there.
URLs added this way stay until removed. WPOrgSidebar triggers wporgsidebar:panel once a panel loads, for the
buttons to be added.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Match the Plugin Admins team's name, and log when a team isn't found.
- Keep other modules' tags and snooze icons out of the hidden subject.
- Mark only the previous conversations core shows in the sidebar.
- Only flag the ownership check when every check ran and none passed.
- Copy each marked subject at its own place in the selection.
- Keep a page's download URL when it can't get a lock, and let tabs settle before refreshing.
- Link the WordPress.org page only for plugins and themes that have one.
- Round short ages up to the next unit, like 1y instead of 12mo.
- Cache the worklist counts of agents who only see assigned conversations.
- Read the worklist members once per page.
- Name close reasons as the plugin directory does, and share the status labels between the panel and the sidebar.
- Update the README's rate limit, and tidy comments and types.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Close the hidden subject span only when it was opened, parsing the subject once per row.
- Keep the statuses of plugins and themes in one table, and label any status in the panel.
- Look up close and rejection reasons through one helper.
- Check whether a plugin is released once per item.
- Reuse is_folder(), inline markCheck(), and merge duplicate CSS rules.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@obenland
obenland force-pushed the fran/freescout-fixes branch from 6e98c2e to e7b339e Compare October 16, 2026 21:28
@obenland
obenland merged commit e7b339e into obenland:add/freescout-plugin-review Oct 16, 2026
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