Skip to content

Conversation

@anubisthejackle
Copy link
Contributor

@anubisthejackle anubisthejackle commented Aug 27, 2024

Summary

This PR implements a new feature to prevent Alley users (and similar "staff" accounts) from appearing as public authors on the frontend of WordPress sites. It addresses #99.

Key aspects of the implementation include:

  • Author archives for filtered accounts (e.g., those with Alley email domains) now return a 404.
  • Author names and links are filtered in traditional authors, Byline Manager, and Co-Authors Plus integrations, replacing them with "Staff" as appropriate.
  • The list of domains and environments affected by this feature can be customized via filters.
  • Feature loading can be restricted by environment type, and this logic is now filterable for greater flexibility.
  • Comprehensive automated tests have been added to ensure the feature works as expected and can be toggled by environment and product context.
  • PHPUnit and test configuration updates were made to support new environment-based logic and exclude integration tests that require unavailable plugins by default.

Notes for reviewers

  • The feature is modular and can be enabled/disabled or filtered by environment.
  • Tests confirm correct behavior across different products and configurations.

Other Information

  • I updated the README.md file for any new/updated features.
  • I updated the CHANGELOG.md file for any new/updated features.

Changelog entries

Added

  • New "Disable Alley Authors" feature to prevent staff user accounts from appearing as public authors.
  • 404 responses for author archives of filtered accounts.
  • Author name and link filtering for traditional authors, Byline Manager, and Co-Authors Plus.
  • Customizable domain and environment filters for feature scope.
  • Filter hooks to control feature loading by environment.
  • PHPUnit config updates and new automated tests for this feature.

Changed

  • Test configuration to support environment-based feature toggling.

Deprecated

Removed

Fixed

Security

@anubisthejackle anubisthejackle marked this pull request as ready for review July 30, 2025 15:17
@anubisthejackle anubisthejackle requested a review from a team as a code owner July 30, 2025 15:17
Copy link
Contributor

@renatonascalves renatonascalves left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🌵

@srtfisher srtfisher linked an issue Sep 12, 2025 that may be closed by this pull request
@anubisthejackle anubisthejackle merged commit d86b294 into main Oct 21, 2025
7 of 14 checks passed
@anubisthejackle anubisthejackle deleted the feature/issue-99/prevent-alley-users-from-appearing branch October 21, 2025 15:01
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.

Prevent Alley Users from Appearing On the Site

3 participants