Adding Feide ID to guest registation page if available

frontend/public/locales/en/common.json

@@ -97,5 +97,6 @@
     "yourGuestPeriod": "Your guest period",
     "guestPeriodDescription": "Period registered for your guest role."
   },
-  "yourGuestAccount": "Your guest account"
+  "yourGuestAccount": "Your guest account",
+  "feideId": "Feide ID"
 }

frontend/public/locales/nb/common.json

@@ -96,5 +96,6 @@
     "yourGuestPeriod": "Din gjesteperiode",
     "guestPeriodDescription": "Registrert periode for din gjesterolle."
   },
-  "yourGuestAccount": "Din gjestekonto"
+  "yourGuestAccount": "Din gjestekonto",
+  "feideId": "Feide ID"
 }

frontend/public/locales/nn/common.json

@@ -97,5 +97,6 @@
     "yourGuestPeriod": "Din gjesteperiode",
     "guestPeriodDescription": "Registrert periode for di gjesterolle."
   },
-  "yourGuestAccount": "Din gjestekonto"
+  "yourGuestAccount": "Din gjestekonto",
+  "feideId": "Feide ID"
 }

frontend/src/routes/guest/register/guestDataForm.ts

@@ -16,6 +16,8 @@ export type GuestInviteInformation = {
   role_end: string
   comment?: string
 
+  feide_id?: string
+
   // These fields are in the form, but it is not expected that
   // they are set, with the exception of e-mail, when the guest
   // first follows the invite link

frontend/src/routes/guest/register/index.tsx

@@ -103,6 +103,7 @@ export default function GuestRegister() {
             comment: jsonResponse.role.comments,
 
             email: jsonResponse.person.email,
+            feide_id: jsonResponse.person.feide_id,
             mobile_phone_country_code: countryCode,
             mobile_phone: nationalNumber,
             fnr: jsonResponse.fnr,

frontend/src/routes/guest/register/registerPage.tsx

@@ -158,6 +158,16 @@ const GuestRegisterStep = forwardRef(
                 disabled
               />
 
+              {/* Only show the Feide ID field if the value is present */}
+              {guestData.feide_id && (
+                <TextField
+                  id="feide_id"
+                  label={t('feideId')}
+                  value={guestData.feide_id}
+                  disabled
+                />
+              )}
+
               <Box
                 sx={{
                   display: 'flex',

gregui/api/views/invitation.py

 from enum import Enum
+from typing import Optional
 
 from django.core import exceptions
 from django.db import transaction
@@ -12,7 +13,7 @@ from rest_framework.permissions import AllowAny
 from rest_framework.response import Response
 from rest_framework.views import APIView
 
-from greg.models import Identity, InvitationLink
+from greg.models import Identity, InvitationLink, Person
 from greg.permissions import IsSponsor
 from gregui.api.serializers.guest import GuestRegisterSerializer
 from gregui.api.serializers.invitation import InviteGuestSerializer
@@ -141,19 +142,9 @@ class InvitedGuestView(GenericAPIView):
             else SessionType.FEIDE.value
         )
 
-        try:
-            fnr = person.identities.get(
-                type=Identity.IdentityType.NORWEGIAN_NATIONAL_ID_NUMBER
-            ).value
-        except Identity.DoesNotExist:
-            fnr = None
-
-        try:
-            passport = person.identities.get(
-                type=Identity.IdentityType.PASSPORT_NUMBER
-            ).value
-        except Identity.DoesNotExist:
-            passport = None
+        fnr = self._get_identity_or_none(person, Identity.IdentityType.NORWEGIAN_NATIONAL_ID_NUMBER)
+        passport = self._get_identity_or_none(person, Identity.IdentityType.PASSPORT_NUMBER)
+        feide_id = self._get_identity_or_none(person, Identity.IdentityType.FEIDE_ID)
 
         data = {
             "person": {
@@ -163,6 +154,7 @@ class InvitedGuestView(GenericAPIView):
                 "mobile_phone": person.private_mobile and person.private_mobile.value,
                 "fnr": fnr,
                 "passport": passport,
+                "feide_id": feide_id
             },
             "sponsor": {
                 "first_name": sponsor.first_name,
@@ -247,3 +239,12 @@ class InvitedGuestView(GenericAPIView):
         )
         # Check that there are no other fields filled in
         return number_of_fields_filled_in == len(person_data.keys())
+
+    @staticmethod
+    def _get_identity_or_none(person: Person, identity_type: Identity.IdentityType) -> Optional[str]:
+        try:
+            return person.identities.get(
+                type=identity_type
+            ).value
+        except Identity.DoesNotExist:
+            return None

gregui/authentication/auth_backends.py

@@ -244,6 +244,7 @@ class GregOIDCBackend(ValidatingOIDCBackend):
                     person=person,
                     source=settings.FEIDE_SOURCE,
                     verified=Identity.Verified.AUTOMATIC,
+                    # TODO Should this be set at this stage or when the sponsor approves the identity?
                     verified_at=timezone.now(),
                 )
                 identity.save()