Skip to content

AO3-7286 Fix spamban page header to display username and ID#5599

Open
james-unsworth wants to merge 2 commits intootwcode:masterfrom
james-unsworth:AO3-7286_spamban_page_heading
Open

AO3-7286 Fix spamban page header to display username and ID#5599
james-unsworth wants to merge 2 commits intootwcode:masterfrom
james-unsworth:AO3-7286_spamban_page_heading

Conversation

@james-unsworth
Copy link

Pull Request Checklist

Issue

https://otwarchive.atlassian.net/browse/AO3-7286

Purpose

The spamban confirmation page now displays the username and user ID of the user to be banned.

Credit

James Unsworth (he/him)

@sarken
Copy link
Collaborator

sarken commented Feb 20, 2026

Hi, James Unsworth!

Thank you so much for this pull request. Someone will be along to review it soon.

I've updated the Jira issue status to In Review so no one mistakenly creates a duplicate pull request. If you'd like the ability to comment on, assign, and transition issues in the future, you're welcome to create a Jira account! It makes things a bit easier for us on the organizational side if the Full Name on your Jira account either closely matches the name you'd like us to credit in the release notes or includes it in parentheses, e.g. "Nickname (CREDIT NAME)."

Once you've done that (or if you've already done it -- Jira has been unreliable about showing us new accounts in the admin panel lately), you can either reply here or send an email to otw-coders@transformativeworks.org with your account name and email address and we'll set up the permissions for you.

Thanks again for contributing! If you have any questions, you can contact us at the same email address listed above.

@james-unsworth
Copy link
Author

Thanks! I created a Jira account under the name James Unsworth, email: jamesunsworth00@gmail.com

@Bilka2
Copy link
Contributor

Bilka2 commented Feb 20, 2026

I sent you a Jira invite that should give you the permissions automatically, please let us know if you have any issues signing up!

Copy link
Contributor

@Bilka2 Bilka2 left a comment

Choose a reason for hiding this comment

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

The change looks good, however it would be nice to add a test for this! If you're looking for inspiration, the test added in #5586 is quite similar, but I can explain more if you have questions, just let me know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants