AO3-7408 Request invatations page title adjustment#5753
AO3-7408 Request invatations page title adjustment#5753jmalark wants to merge 1 commit intootwcode:masterfrom
Conversation
|
@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. |
cc0bd5b to
be6a668
Compare
sarken
left a comment
There was a problem hiding this comment.
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") |
There was a problem hiding this comment.
This should also be changed to "Request Invitations" -- and if you could make it translatable like Reviewdog mentioned, we'd appreciate that!
There was a problem hiding this comment.
Ok changed the text and moved it to the en.yml file
97550a5 to
5ae8315
Compare
bfd3936 to
fe2327e
Compare
Pull Request Checklist
as the first thing in your pull request title (e.g.
AO3-1234 Fix thing)until they are reviewed and merged before creating new pull requests.
Issue
https://otwarchive.atlassian.net/browse/AO3-7408
Purpose
Corrects the browser page title for the Request Invitations page
Credit
Jesse Malark he/him