Skip to content

Add Ban Hammer and Stop Forum Spam disapprove screenshot recipes - #66

Merged
bonelifer merged 1 commit into
mainfrom
feature/banhammer-sfs-disapprove-shots
Nov 10, 2026
Merged

bonelifer merged 1 commit into
mainfrom
feature/banhammer-sfs-disapprove-shots

Conversation

@bonelifer

Copy link
Copy Markdown
Contributor

Adds two screenshot recipes to phpbb-test-board/screenshots.py.

  • Ban Hammer (phpbbmodders/banhammer, new): turns the ban list on, bans two seeded members with phpBB's user_ban() (never members of a group with a moderator role), and captures the ACP settings page and the ban list page.
  • Stop Forum Spam: adds the Report the poster to Stop Forum Spam option on the MCP disapprove form. It puts one reply in the moderation queue, presses Disapprove and captures the form. The option needs the mcp_approve_reason_after template event from phpBB 3.3.20-RC1, so on an older board the shot is skipped with a note instead of failing.

Claude ran both on a phpBB 3.3.19 board built with setup-board.sh and seed-forum's standard fixtures: Ban Hammer saved 2 of 2, and Stop Forum Spam saved its 2 existing shots and skipped the disapprove shot as intended. The disapprove capture itself has not run yet, because no released phpBB has the event.

🤖 Generated with Claude Code


Investigated and written by Claude on behalf of William Jacoby (bonelifer).

Ban Hammer: the ACP settings with the ban list switch and the ban list
page, with two seeded members banned through user_ban(). Stop Forum Spam:
the report option on the MCP disapprove form, skipped with a note on
phpBB older than 3.3.20, which has no mcp_approve_reason_after event.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@bonelifer
bonelifer merged commit 7597c05 into main Nov 10, 2026
1 check passed
@bonelifer
bonelifer deleted the feature/banhammer-sfs-disapprove-shots branch November 10, 2026 18:17
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.

1 participant