teecup/app/routers/notifications.py
Erol Haagenrud 17fac05ddc Alt kompilerer rent. Oppsummering av denne runden:
1. WHS Rule 3.1b — ferdig, testet (117/117), rullet ut... nei, venter fortsatt (se under).
2. round.tee_name_snapshot-synk — ferdig, scratch-verifisert.
3. Optimistisk versjonssjekk for samtidig hull-redigering — ferdig bygget, og jeg fant og rettet en reell UX-bug underveis (en fullskjerm-feilvisning i stedet for en liten banner) som ellers ville gjort funksjonen verre enn ingenting.

Alt er dokumentert i ADR-056/057 og CHANGELOG punkt 65–66.

Deploy — rekkefølgen har betydning (migrasjonen må kjøre først, siden ny kode leser version-kolonnen):

# 1. Migrasjon mot ekte teecup_db
docker exec teeoff_db psql -U teeoff_admin -d teecup_db -f /opt/teecup/062_round_hole_version.sql

# 2. Backend
docker compose build teecup_api
docker compose up -d teecup_api

# 3. Frontend
docker compose build teecup_frontend
docker compose up -d teecup_frontend
Og fortsatt uavklart fra tidligere: datakorreksjonen for din spesifikke runde

UPDATE round SET tee_name_snapshot = '55'
WHERE id = '520f9681-6e7f-49bf-8504-9ab223874285';
2026-08-10 18:34:31 +02:00

282 lines
10 KiB
Python

"""
In-app varslingssenter (FEATURE_BACKLOG.md "Varsler"-runden, 2026-07-25) +
e-post-fallback per type (2026-07-28 oppfølging, migrasjon 033) + push til
telefonens OS (2026-07-28, migrasjon 037, se app/push.py). Push har -- ulikt
e-post -- INGEN egen per-type opt-in: selve det å abonnere (gi
nettleser-tillatelse) ER samtykket, en abonnert enhet får push for ALLE
typer create_notification() skriver.
Eid av BRUKER (mottaker), ikke organisasjon -- plain_connection(), samme
mønster som personlig profil/HCP-historikk/venner. `create_notification()`
er den eneste skrivevegen inn -- kalt fra andre routere (friends.py,
rounds.py) ved konkrete hendelser, ikke noe generisk event-system.
Trygg standard (samme filosofi som visibility_mode='private' ellers i
appen): en bruker som ikke har gjort noe valg får ALDRI e-post for noen
type -- kun `user_notification_email_pref`-raden EN BRUKER selv setter
(via PUT /notifications/email-prefs) slår e-post PÅ for akkurat den typen.
E-post-utsendingen skjer INNI `create_notification()` selv (ikke i hvert
enkelt kallsted) -- driftsfeil i utsendingen skal ALDRI hindre selve
in-app-varselet fra å bli opprettet, samme mønster (SMTP_CONFIGURED-sjekk
+ try/except + traceback.print_exc()) som all annen e-post i appen
(app/routers/auth.py, organizations.py).
"""
from __future__ import annotations
import traceback
from typing import Literal
from fastapi import APIRouter, Depends
from pydantic import BaseModel
from ..auth import CurrentUser, get_current_user
from ..config import settings
from ..db import plain_connection
from ..email import send_notification_email
from ..errors import app_error
from ..push import send_push_to_user
router = APIRouter(tags=["notifications"])
NotificationType = Literal["friend", "tournament", "round", "result"]
_NOTIFICATION_TYPES: tuple[NotificationType, ...] = ("friend", "tournament", "round", "result")
_PUSH_TITLES: dict[str, str] = {
"friend": "Venner",
"round": "Runde",
"result": "Resultat",
"tournament": "Turnering",
}
async def create_notification(
conn, *, user_id: str, type: NotificationType, message: str, link_path: str, allow_email: bool = True
) -> None:
"""Kalles fra andre routere sin egen `plain_connection()`/transaksjon --
tar en allerede-åpen `conn`, åpner ikke en egen. Sender i tillegg en
e-post-fallback hvis (og kun hvis) mottakeren selv har valgt inn for
akkurat DENNE typen, OG et push-varsel til enhver enhet mottakeren har
abonnert (uavhengig av e-post-valget -- se _push_for_notification).
`allow_email=False` (2026-08-10) lar en enkelt kallsted permanent
utelukke e-post for akkurat DEN hendelsen, selv om brukeren har krysset
av for typen generelt -- brukt av "venn startet en runde du kan følge"
i rounds.py, som deler `type="round"` med "lagt til som medspiller"
(som fortsatt skal kunne e-postes). In-app-varselet og push forblir
uendret uansett -- kun selve e-postutsendingen droppes."""
await conn.execute(
"INSERT INTO notification (user_id, type, message, link_path) VALUES ($1, $2, $3, $4)",
user_id,
type,
message,
link_path,
)
await _push_for_notification(conn, user_id=user_id, type=type, message=message, link_path=link_path)
if not allow_email:
return
wants_email = await conn.fetchval(
"SELECT EXISTS(SELECT 1 FROM user_notification_email_pref WHERE user_id = $1 AND type = $2)",
user_id,
type,
)
if not wants_email:
return
recipient = await conn.fetchrow(
"SELECT email, preferred_locale FROM app_user WHERE id = $1 AND email IS NOT NULL",
user_id,
)
if recipient is None:
return
if settings.SMTP_CONFIGURED:
try:
await send_notification_email(recipient["email"], message, link_path, recipient["preferred_locale"])
except Exception:
# Se modul-docstring: driftsfeil i utsendingen skal aldri
# hindre selve in-app-varselet, som allerede er skrevet over.
traceback.print_exc()
elif settings.DEV_LOG_MAGIC_LINKS:
print(f"[DEV] Varsel-e-post til {recipient['email']} ({type}): {message}", flush=True)
async def _push_for_notification(conn, *, user_id: str, type: NotificationType, message: str, link_path: str) -> None:
"""Kalt fra create_notification() -- egen liten funksjon KUN for at
push (uavhengig av e-post-preferansen over) alltid forsøkes for enhver
abonnert enhet. Feiler aldri synlig for kalleren, se app/push.py."""
try:
await send_push_to_user(conn, user_id, _PUSH_TITLES.get(type, "TeeCup"), message, link_path)
except Exception:
traceback.print_exc()
class NotificationEmailPrefsOut(BaseModel):
types: list[str]
class NotificationEmailPrefsUpdate(BaseModel):
types: list[NotificationType]
@router.get("/notifications/email-prefs", response_model=NotificationEmailPrefsOut)
async def get_email_prefs(user: CurrentUser = Depends(get_current_user)) -> NotificationEmailPrefsOut:
async with plain_connection() as conn:
rows = await conn.fetch(
"SELECT type FROM user_notification_email_pref WHERE user_id = $1",
user.user_id,
)
return NotificationEmailPrefsOut(types=[r["type"] for r in rows])
@router.put("/notifications/email-prefs", response_model=NotificationEmailPrefsOut)
async def set_email_prefs(
body: NotificationEmailPrefsUpdate, user: CurrentUser = Depends(get_current_user)
) -> NotificationEmailPrefsOut:
"""Full-erstatning, samme mønster som PUT /friends/{id}/categories --
sender med en tom liste slår e-post AV for alle typer igjen."""
types = sorted(set(body.types))
async with plain_connection() as conn, conn.transaction():
await conn.execute("DELETE FROM user_notification_email_pref WHERE user_id = $1", user.user_id)
for t in types:
await conn.execute(
"INSERT INTO user_notification_email_pref (user_id, type) VALUES ($1, $2)",
user.user_id,
t,
)
return NotificationEmailPrefsOut(types=types)
class NotificationOut(BaseModel):
id: str
type: str
message: str
link_path: str
created_at: str
read_at: str | None
@router.get("/notifications", response_model=list[NotificationOut])
async def list_notifications(user: CurrentUser = Depends(get_current_user)) -> list[NotificationOut]:
async with plain_connection() as conn:
rows = await conn.fetch(
"""
SELECT id::text AS id, type, message, link_path, created_at, read_at
FROM notification WHERE user_id = $1
ORDER BY created_at DESC
LIMIT 100
""",
user.user_id,
)
return [
NotificationOut(
id=r["id"],
type=r["type"],
message=r["message"],
link_path=r["link_path"],
created_at=r["created_at"].isoformat(),
read_at=r["read_at"].isoformat() if r["read_at"] else None,
)
for r in rows
]
@router.get("/notifications/unread-count")
async def unread_count(user: CurrentUser = Depends(get_current_user)) -> dict[str, int]:
async with plain_connection() as conn:
count = await conn.fetchval(
"SELECT COUNT(*) FROM notification WHERE user_id = $1 AND read_at IS NULL",
user.user_id,
)
return {"count": count}
@router.post("/notifications/{notification_id}/read")
async def mark_read(notification_id: str, user: CurrentUser = Depends(get_current_user)) -> dict[str, bool]:
async with plain_connection() as conn:
await conn.execute(
"UPDATE notification SET read_at = now() WHERE id = $1 AND user_id = $2 AND read_at IS NULL",
notification_id,
user.user_id,
)
return {"ok": True}
@router.post("/notifications/read-all")
async def mark_all_read(user: CurrentUser = Depends(get_current_user)) -> dict[str, bool]:
async with plain_connection() as conn:
await conn.execute(
"UPDATE notification SET read_at = now() WHERE user_id = $1 AND read_at IS NULL",
user.user_id,
)
return {"ok": True}
# ---------------------------------------------------------------------------
# Push-varsler til telefonens OS (Web Push/VAPID, migrasjon 037). Offentlig
# nøkkel er nettopp det -- offentlig, ingen auth nødvendig for GET-en (samme
# begrunnelse som ethvert annet "her er min offentlige nøkkel"-endepunkt).
# Abonner/avbryt krever derimot en ekte sesjon -- abonnementet er alltid
# knyttet til AKKURAT den innloggede brukeren.
# ---------------------------------------------------------------------------
class VapidPublicKeyOut(BaseModel):
public_key: str | None
configured: bool
@router.get("/push/vapid-public-key", response_model=VapidPublicKeyOut)
async def get_vapid_public_key() -> VapidPublicKeyOut:
return VapidPublicKeyOut(public_key=settings.VAPID_PUBLIC_KEY, configured=settings.PUSH_CONFIGURED)
class PushSubscriptionKeys(BaseModel):
p256dh: str
auth: str
class PushSubscriptionCreate(BaseModel):
endpoint: str
keys: PushSubscriptionKeys
@router.post("/push/subscribe")
async def subscribe_push(
body: PushSubscriptionCreate, user: CurrentUser = Depends(get_current_user)
) -> dict[str, bool]:
if not settings.PUSH_CONFIGURED:
raise app_error(400, "VALIDATION_FAILED", "Push er ikke konfigurert på serveren.")
async with plain_connection() as conn:
# Samme endpoint kan i prinsippet re-abonneres (f.eks. nøklene
# rotert av nettleseren) -- oppdater i stedet for å feile på
# unik-constrainten.
await conn.execute(
"""
INSERT INTO push_subscription (user_id, endpoint, p256dh, auth)
VALUES ($1, $2, $3, $4)
ON CONFLICT (endpoint) DO UPDATE SET user_id = $1, p256dh = $3, auth = $4
""",
user.user_id,
body.endpoint,
body.keys.p256dh,
body.keys.auth,
)
return {"ok": True}
class PushUnsubscribe(BaseModel):
endpoint: str
@router.delete("/push/subscribe")
async def unsubscribe_push(
body: PushUnsubscribe, user: CurrentUser = Depends(get_current_user)
) -> dict[str, bool]:
async with plain_connection() as conn:
await conn.execute(
"DELETE FROM push_subscription WHERE endpoint = $1 AND user_id = $2",
body.endpoint,
user.user_id,
)
return {"ok": True}