Add display name (first_name) shown instead of username on user-facing UI

allauth_oauth/signals.py

@@ -29,6 +29,11 @@ def sync_user_fields(sender, request, sociallogin, **kwargs):
         user.username = username
         changed = True
 
+    display_name = data.get("name") or data.get("preferred_username")
+    if display_name and user.first_name != display_name:
+        user.first_name = display_name
+        changed = True
+
     staff_groups = settings.OAUTH_STAFF_GROUPS
     if staff_groups:
         oauth_groups = set(data.get("groups", []))

photo21/forms.py

@@ -8,6 +8,12 @@ from django.utils.translation import gettext_lazy as _
 
 
 class CustomSignupForm(SignupForm):
+    first_name = forms.CharField(
+        label=_("Display name"),
+        max_length=150,
+        required=True,
+    )
+
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
 
@@ -16,6 +22,12 @@ class CustomSignupForm(SignupForm):
             "Please enter a valid email address ending with `@ens-rennes.fr`"
         )
 
+    def save(self, request):
+        user = super().save(request)
+        user.first_name = self.cleaned_data["first_name"]
+        user.save()
+        return user
+
     def clean_email(self):
         """
         Check that the email address ends with a trusted domain.

photo21/templates/base.html

@@ -132,7 +132,7 @@ SPDX-License-Identifier: GPL-3.0-or-later
     <footer>
         <p class="small text-center text-muted mt-1">
             {% if request.user.is_authenticated %}
-                {% trans "Connected as" %} <code>{{ request.user.username }}</code> &middot;
+                {% trans "Connected as" %} <code>{{ request.user.first_name|default:request.user.username }}</code> &middot;
             {% endif %}
             <a class="text-reset" href="https://git.sinfonie.org/sinfonie/photo26">{% trans "Source code" %}</a>
         </p>

photologue/templates/photologue/gallery_detail.html

@@ -85,7 +85,7 @@ SPDX-License-Identifier: GPL-3.0-or-later
             <li class="nav-item">
                 {% url 'photologue:pl-gallery-owner' slug=gallery.slug owner=owner.id as url %}
                 <a class="nav-link {% if request.path_info == url %}active{% endif %}" href="{{ url }}">
-                    {% if owner.get_full_name %}{{ owner.get_full_name }}{% else %}{{ owner.username }}{% endif %}
+                    {{ owner.first_name|default:owner.username }}
                 </a>
             </li>
             {% endfor %}
@@ -114,7 +114,7 @@ SPDX-License-Identifier: GPL-3.0-or-later
            data-height="{{ item.thumbnail_height|default:1 }}">
             <img src="{{ item.get_thumbnail_url }}" data-lazy="{{ item.get_thumbnail_url }}"
                  class="{% if not item.is_public %}photo-private{% endif %}"
-                 alt="{{ item.title }}{% if not item.is_video and item.date_taken %} - {{ item.date_taken|date }} {{ item.date_taken|time }}{% endif %}{% if item.owner.get_full_name %} - {{ item.owner.get_full_name }}{% else %} - {{ item.owner.username }}{% endif %}{% if not item.is_video and item.license %} - {{ item.license }}{% endif %}{% if not item.is_public %} - !PRIVATE!{% endif %}">
+                 alt="{{ item.title }}{% if not item.is_video and item.date_taken %} - {{ item.date_taken|date }} {{ item.date_taken|time }}{% endif %} - {{ item.owner.first_name|default:item.owner.username }}{% if not item.is_video and item.license %} - {{ item.license }}{% endif %}{% if not item.is_public %} - !PRIVATE!{% endif %}">
         </a>
         {% endfor %}
     </div>