Skip to content

AO3-7222 Check can_change_username to show Rename button for admin - #5989

Open
Panekd wants to merge 2 commits into
otwcode:masterfrom
Panekd:AO3-7222
Open

AO3-7222 Check can_change_username to show Rename button for admin#5989
Panekd wants to merge 2 commits into
otwcode:masterfrom
Panekd:AO3-7222

Conversation

@Panekd

@Panekd Panekd commented Aug 19, 2026

Copy link
Copy Markdown

Pull Request Checklist

Issue

AO3-7222

Purpose

Add a check of can_rename_user to show Rename button on user administration page so it is only shown to admins with permission to rename users

Testing Instructions

See Jira

Credit

Rose Hardy - she/her

@sarken

sarken commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator

Hi, Rose Hardy!

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

In the meantime, 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.

@danderson danderson left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I'm just an onlooker, but the change looks good to me.

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