-
Notifications
You must be signed in to change notification settings - Fork 37
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
Use preferred names for templates in Tock #1167
Conversation
Using django's provided `user.get_full_name` we now render full name, configurable in the admin, for display throughout Tock.
Also removing duplicated test `hours.test.test_views.test_project_timeline`
Codecov Report
@@ Coverage Diff @@
## main #1167 +/- ##
==========================================
- Coverage 88.93% 88.22% -0.72%
==========================================
Files 51 51
Lines 2097 2097
==========================================
- Hits 1865 1850 -15
- Misses 232 247 +15
Continue to review full report at Codecov.
|
* main: Remove requirements.txt
Since we're using UserData to get a display name, let's get them all on our first query instead each one individually.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Big round of 👏 for @Sgtpluck. Great job, especially as you're still learning your way around Django.
Description
For #1027, we now render preferred names (First + Last) -- configurable via the admin panel -- for all pages in Tock.
This does not include changes to the API endpoints or data exports.
There's a handful of users (~9) who don't have a First/Last name value set -- We could handled with a bit more code here, rendering
username
as a fallback OR updating these users in the admin.We should also update the Tock change form so folks can request a name update...OR create a form so folks can update it themselves