Skip to content

Commit 0b3b53b

Browse files
committed
update username logic
1 parent 5e5a34e commit 0b3b53b

File tree

5 files changed

+24
-12
lines changed

5 files changed

+24
-12
lines changed

backend/api/context.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -48,18 +48,18 @@ async def get_context(
4848
if not db_user:
4949
db_user = User(
5050
id=user_id,
51-
name=username,
51+
username=username,
5252
firstname=firstname,
5353
lastname=lastname,
5454
title="User",
5555
)
5656
session.add(db_user)
5757
elif (
58-
db_user.name != username
58+
db_user.username != username
5959
or db_user.firstname != firstname
6060
or db_user.lastname != lastname
6161
):
62-
db_user.name = username
62+
db_user.username = username
6363
db_user.firstname = firstname
6464
db_user.lastname = lastname
6565

backend/api/types/user.py

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,12 +11,18 @@
1111
@strawberry.type
1212
class UserType:
1313
id: strawberry.ID
14-
name: str
14+
username: str
1515
firstname: str | None
1616
lastname: str | None
1717
title: str | None
1818
avatar_url: str | None
1919

20+
@strawberry.field
21+
def name(self) -> str:
22+
if self.firstname and self.lastname:
23+
return f"{self.firstname} {self.lastname}"
24+
return self.username
25+
2026
@strawberry.field
2127
async def tasks(
2228
self,

backend/auth.py

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,13 @@
22
from typing import Any, Optional
33

44
import requests
5-
from config import CLIENT_ID, FRONTEND_CLIENT_ID, ISSUER_URI, LOGGER, PUBLIC_ISSUER_URI
5+
from config import (
6+
CLIENT_ID,
7+
FRONTEND_CLIENT_ID,
8+
ISSUER_URI,
9+
LOGGER,
10+
PUBLIC_ISSUER_URI,
11+
)
612
from fastapi import Request
713
from fastapi.responses import RedirectResponse
814
from jose import jwk, jwt
@@ -95,7 +101,7 @@ def verify_token(token: str) -> dict:
95101
elif aud is None:
96102
aud = []
97103

98-
if azp and azp == CLIENT_ID or azp == FRONTEND_CLIENT_ID:
104+
if (azp and azp == CLIENT_ID) or azp == FRONTEND_CLIENT_ID:
99105
return payload
100106

101107
if CLIENT_ID in aud or FRONTEND_CLIENT_ID in aud:

backend/database/migrations/versions/92aad7cc3186_initial_models.py renamed to backend/database/migrations/versions/ac24ef41bd41_.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
1-
"""Initial models
1+
"""empty message
22
3-
Revision ID: 92aad7cc3186
3+
Revision ID: ac24ef41bd41
44
Revises:
5-
Create Date: 2025-12-01 02:36:26.250686
5+
Create Date: 2025-12-12 19:00:58.216109
66
77
"""
88
from typing import Sequence, Union
@@ -12,7 +12,7 @@
1212

1313

1414
# revision identifiers, used by Alembic.
15-
revision: str = '92aad7cc3186'
15+
revision: str = 'ac24ef41bd41'
1616
down_revision: Union[str, Sequence[str], None] = None
1717
branch_labels: Union[str, Sequence[str], None] = None
1818
depends_on: Union[str, Sequence[str], None] = None
@@ -41,7 +41,7 @@ def upgrade() -> None:
4141
)
4242
op.create_table('users',
4343
sa.Column('id', sa.String(), nullable=False),
44-
sa.Column('name', sa.String(), nullable=False),
44+
sa.Column('username', sa.String(), nullable=False),
4545
sa.Column('firstname', sa.String(), nullable=True),
4646
sa.Column('lastname', sa.String(), nullable=True),
4747
sa.Column('title', sa.String(), nullable=True),

backend/database/models/user.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ class User(Base):
1919
primary_key=True,
2020
default=lambda: str(uuid.uuid4()),
2121
)
22-
name: Mapped[str] = mapped_column(String)
22+
username: Mapped[str] = mapped_column(String, nullable=False)
2323
firstname: Mapped[str | None] = mapped_column(String, nullable=True)
2424
lastname: Mapped[str | None] = mapped_column(String, nullable=True)
2525
title: Mapped[str | None] = mapped_column(String, nullable=True)

0 commit comments

Comments
 (0)