Skip to content

AO3-7408 Request invatations page title adjustment#5753

Open
jmalark wants to merge 1 commit intootwcode:masterfrom
jmalark:AO3-7408-request-invatations-page-title-adjustment
Open

AO3-7408 Request invatations page title adjustment#5753
jmalark wants to merge 1 commit intootwcode:masterfrom
jmalark:AO3-7408-request-invatations-page-title-adjustment

Conversation

@jmalark
Copy link
Copy Markdown
Contributor

@jmalark jmalark commented Apr 22, 2026

Pull Request Checklist

Issue

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

Purpose

Corrects the browser page title for the Request Invitations page

Credit

Jesse Malark he/him

@jmalark
Copy link
Copy Markdown
Contributor Author

jmalark commented Apr 22, 2026

@sarken It didn't look like we were doing any RSpec tests for @page_subtitle getting set to hard coded strings (only ones based on variables) so I didn't add anything to user_invite_requests_controller_spec.rb. If you want me to throw a test in there for this, just let me know.

@jmalark jmalark force-pushed the AO3-7408-request-invatations-page-title-adjustment branch 3 times, most recently from cc0bd5b to be6a668 Compare April 22, 2026 22:58
Copy link
Copy Markdown
Collaborator

@sarken sarken left a comment

Choose a reason for hiding this comment

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

Thanks for working on this!

Regarding tests, we usually use Cucumber for the page titles -- you can find a bunch of examples if you search for either "I should see the page title" or "the page title should include."

In this case, adding Then the page title should include "Request Invitations" after line 9 in the first scenario in other_a/invite_request.feature should work.

# GET /user_invite_requests/new.xml
def new
@page_title = ts('New User Invitation Request')
@page_subtitle = ts("New User Invitation Request")
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This should also be changed to "Request Invitations" -- and if you could make it translatable like Reviewdog mentioned, we'd appreciate that!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ok changed the text and moved it to the en.yml file

@jmalark jmalark force-pushed the AO3-7408-request-invatations-page-title-adjustment branch from bfd3936 to fe2327e Compare April 23, 2026 01:26
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.

2 participants