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

Update to titles for users #250

Merged
merged 1 commit into from Feb 1, 2021
Merged

Update to titles for users #250

merged 1 commit into from Feb 1, 2021

Conversation

GRyall
Copy link
Contributor

@GRyall GRyall commented Aug 24, 2020

Fixes #176. On user edit and registration pages:

  • Add title 'Mx'
  • Add option for empty title
  • Update Gocdb schema to allow empty string titles (simpler to
    maintain than having a null option on the drop down)
  • Reorder titles to be in alphabetical order

I have tested the edit page. I have not been able to test the registration page (though the code is very similar). It should be tested before merging.

On user edit and registration pages:
* Add title 'Mx'
* Add option for empty title
* Update Gocdb schema to allow empty string titles (simpler to
maintain than having a null option on the drop down)
* Reorder titles to be in alphabetical order
Copy link
Member

@tofu-rocketry tofu-rocketry left a comment

Choose a reason for hiding this comment

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

🏳️‍🌈

@gregcorbett gregcorbett self-assigned this Aug 25, 2020
@gregcorbett gregcorbett added this to the 5.8.0 milestone Aug 25, 2020
Copy link
Member

@gregcorbett gregcorbett left a comment

Choose a reason for hiding this comment

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

Registering as a new user works as expected, as does editing a user.

@gregcorbett gregcorbett merged commit e2a783c into GOCDB:dev Feb 1, 2021
@GRyall GRyall deleted the regPage branch February 2, 2021 11:17
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.

Title on user registration page
3 participants