Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AO3-6579 Set @page_title to Create Account not New Registration #4675

Merged
merged 1 commit into from
Dec 4, 2023

Conversation

helpotters
Copy link
Contributor

@helpotters helpotters commented Nov 29, 2023

The heading in the New User Registrations page says "Create Account", however, the browser tab says "New Registration". In the controller's :new action, I added @page_title = 'Create Account' to make the browser tab match.

Note: This differs from setting @title, which would change the heading.

In regards to implementation, I tried following the note within the Jira issue:

This should be set in the new action of the Users::Registrations controller.

Pull Request Checklist

Issue

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

Purpose

Match the browser tab @page_title with the Heading @title.

Testing Instructions

Note: Looking for confirmation in regards to how I set the variable. I used a normal string with 'Create Account', rather than pulling it from somewhere like @page_title = ts('New User Invitation Request'). Please confirm if this is okay.

  • Allow for new Account Creation in development
    (I just temporarily removed if !AdminSetting.current.account_creation_enabled? in users/registrations_controller.rb#66.
  • Go to the /signup path (127.0.0.1:3000/signup, depending on your setup).
  • Check your browser tab and ensure that it says "Create Account" instead of "New Registration" (or (development) Create Account).

Credit

Paul Lemus (he/him)

The heading in New User Registrations page says "Create Account",
however the browser tab said "New Registration". In the new action of
the controller, I added @page_title = 'Create Account' to make the
browser tab match.

Note: This is different from setting @title, which would change the
heading.
@helpotters
Copy link
Contributor Author

helpotters commented Nov 29, 2023

Resolved!
I'm also not on the volunteer team, so I don't have the Jira permissions to follow through on the After Submitting a PR.

Let me know if this is okay. It is currently still labeled as an open issue.

@sarken
Copy link
Collaborator

sarken commented Nov 30, 2023

Hi, Paul!

Thank you so much for this pull request -- what you've done is just fine! Someone will be along to review it soon.

In the meantime, I've assigned the Jira issue to you and set its status to In Review, so no one will mistakenly create a duplicate pull request. I've also given your account permission to comment on, assign, and transition issues in the future.

Thanks again for contributing! If you have any questions, you can contact us at otw-coders@transformativeworks.org.

@sarken sarken merged commit 0cbe3c3 into otwcode:master Dec 4, 2023
25 checks passed
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