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

Require User.fullname to be populated #4699

Open
mikerkelly opened this issue Oct 22, 2024 · 0 comments
Open

Require User.fullname to be populated #4699

mikerkelly opened this issue Oct 22, 2024 · 0 comments
Labels
deck-scrubbing Tech debt or other between-initiative tidy-up work

Comments

@mikerkelly
Copy link
Contributor

User.fullname is a TextField that defaults to an empty string. There are 27 User on the live server with fullname == ''. If we can manually fill in that field for each that would allow us to enforce fullname being populated in both the model and views/forms (non-null and non-empty?). That could allow us to be rid of the User.name property which uses fullname but falls back to username if fullname is empty. This would simplify the User model and populated the DB with better data for managing or viewing User instances.

@mikerkelly mikerkelly added the deck-scrubbing Tech debt or other between-initiative tidy-up work label Oct 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deck-scrubbing Tech debt or other between-initiative tidy-up work
Projects
None yet
Development

No branches or pull requests

1 participant