feat(notifications): durable per-user inbox (P1) + maintenance schema
Add a server-backed notification center that coexists with the client-side transient bell: a per-user `notifications` table (type/title/body/data JSON/ read/dismissed), a `/api/me/notifications` CRUD API (list, unread_count, read, read_all, dismiss, clear), and a left-nav inbox module with a live unread badge polled via useLiveQuery. Known types render trilingual text from type+data (English stored text is the fallback); read rows are trimmed past a soft cap. Also adds the schema the maintenance job builds on: videos.list?part=status columns (embeddable/privacy_status/upload_status) and the validation lifecycle columns (last_checked_at, unavailable_since, unavailable_reason). Page-id validation is centralized in one PAGES source of truth (isPage) so the new page survives reload without a second allowlist to keep in sync.
This commit is contained in:
parent
74d46f61eb
commit
3ae42409b3
14 changed files with 649 additions and 22 deletions
43
backend/alembic/versions/0016_video_maintenance.py
Normal file
43
backend/alembic/versions/0016_video_maintenance.py
Normal file
|
|
@ -0,0 +1,43 @@
|
||||||
|
"""video maintenance / validation columns
|
||||||
|
|
||||||
|
Revision ID: 0016_video_maintenance
|
||||||
|
Revises: 0015_scheduler_settings
|
||||||
|
Create Date: 2026-06-18
|
||||||
|
|
||||||
|
Adds the columns the maintenance/validation job needs to detect and retire unplayable
|
||||||
|
videos: videos.list?part=status fields (embeddable / privacy_status / upload_status) plus
|
||||||
|
the validation lifecycle (last_checked_at for rolling re-validation; unavailable_since +
|
||||||
|
unavailable_reason to flag a video as currently unplayable — hidden from the feed
|
||||||
|
immediately, hard-deleted after a grace period). Purely additive; all columns nullable.
|
||||||
|
"""
|
||||||
|
from typing import Sequence, Union
|
||||||
|
|
||||||
|
import sqlalchemy as sa
|
||||||
|
from alembic import op
|
||||||
|
|
||||||
|
revision: str = "0016_video_maintenance"
|
||||||
|
down_revision: Union[str, None] = "0015_scheduler_settings"
|
||||||
|
branch_labels: Union[str, Sequence[str], None] = None
|
||||||
|
depends_on: Union[str, Sequence[str], None] = None
|
||||||
|
|
||||||
|
|
||||||
|
def upgrade() -> None:
|
||||||
|
op.add_column("videos", sa.Column("embeddable", sa.Boolean(), nullable=True))
|
||||||
|
op.add_column("videos", sa.Column("privacy_status", sa.String(length=16), nullable=True))
|
||||||
|
op.add_column("videos", sa.Column("upload_status", sa.String(length=16), nullable=True))
|
||||||
|
op.add_column("videos", sa.Column("last_checked_at", sa.DateTime(timezone=True), nullable=True))
|
||||||
|
op.add_column("videos", sa.Column("unavailable_since", sa.DateTime(timezone=True), nullable=True))
|
||||||
|
op.add_column("videos", sa.Column("unavailable_reason", sa.String(length=24), nullable=True))
|
||||||
|
op.create_index("ix_videos_last_checked_at", "videos", ["last_checked_at"])
|
||||||
|
op.create_index("ix_videos_unavailable_since", "videos", ["unavailable_since"])
|
||||||
|
|
||||||
|
|
||||||
|
def downgrade() -> None:
|
||||||
|
op.drop_index("ix_videos_unavailable_since", table_name="videos")
|
||||||
|
op.drop_index("ix_videos_last_checked_at", table_name="videos")
|
||||||
|
op.drop_column("videos", "unavailable_reason")
|
||||||
|
op.drop_column("videos", "unavailable_since")
|
||||||
|
op.drop_column("videos", "last_checked_at")
|
||||||
|
op.drop_column("videos", "upload_status")
|
||||||
|
op.drop_column("videos", "privacy_status")
|
||||||
|
op.drop_column("videos", "embeddable")
|
||||||
59
backend/alembic/versions/0017_notifications.py
Normal file
59
backend/alembic/versions/0017_notifications.py
Normal file
|
|
@ -0,0 +1,59 @@
|
||||||
|
"""per-user notification inbox (phase 1)
|
||||||
|
|
||||||
|
Revision ID: 0017_notifications
|
||||||
|
Revises: 0016_video_maintenance
|
||||||
|
Create Date: 2026-06-18
|
||||||
|
|
||||||
|
Adds the durable, server-backed per-user notification table behind the inbox module. The
|
||||||
|
JSON payload column is named `data` (SQLAlchemy's declarative Base reserves `metadata`).
|
||||||
|
First producer is the maintenance job's batched "videos removed" notice. Coexists with the
|
||||||
|
existing client-side transient bell.
|
||||||
|
"""
|
||||||
|
from typing import Sequence, Union
|
||||||
|
|
||||||
|
import sqlalchemy as sa
|
||||||
|
from alembic import op
|
||||||
|
|
||||||
|
revision: str = "0017_notifications"
|
||||||
|
down_revision: Union[str, None] = "0016_video_maintenance"
|
||||||
|
branch_labels: Union[str, Sequence[str], None] = None
|
||||||
|
depends_on: Union[str, Sequence[str], None] = None
|
||||||
|
|
||||||
|
|
||||||
|
def upgrade() -> None:
|
||||||
|
op.create_table(
|
||||||
|
"notifications",
|
||||||
|
sa.Column("id", sa.Integer(), primary_key=True),
|
||||||
|
sa.Column(
|
||||||
|
"user_id",
|
||||||
|
sa.Integer(),
|
||||||
|
sa.ForeignKey("users.id", ondelete="CASCADE"),
|
||||||
|
nullable=False,
|
||||||
|
),
|
||||||
|
sa.Column("type", sa.String(length=32), nullable=False),
|
||||||
|
sa.Column("title", sa.String(length=255), nullable=False),
|
||||||
|
sa.Column("body", sa.Text(), nullable=True),
|
||||||
|
sa.Column("data", sa.JSON(), nullable=True),
|
||||||
|
sa.Column("read", sa.Boolean(), nullable=False, server_default="false"),
|
||||||
|
sa.Column("dismissed", sa.Boolean(), nullable=False, server_default="false"),
|
||||||
|
sa.Column(
|
||||||
|
"created_at", sa.DateTime(timezone=True), server_default=sa.func.now()
|
||||||
|
),
|
||||||
|
)
|
||||||
|
op.create_index("ix_notifications_user_id", "notifications", ["user_id"])
|
||||||
|
op.create_index("ix_notifications_type", "notifications", ["type"])
|
||||||
|
op.create_index("ix_notifications_read", "notifications", ["read"])
|
||||||
|
op.create_index("ix_notifications_created_at", "notifications", ["created_at"])
|
||||||
|
# Fast unread-badge count + inbox listing per user.
|
||||||
|
op.create_index(
|
||||||
|
"ix_notifications_user_read", "notifications", ["user_id", "read"]
|
||||||
|
)
|
||||||
|
|
||||||
|
|
||||||
|
def downgrade() -> None:
|
||||||
|
op.drop_index("ix_notifications_user_read", table_name="notifications")
|
||||||
|
op.drop_index("ix_notifications_created_at", table_name="notifications")
|
||||||
|
op.drop_index("ix_notifications_read", table_name="notifications")
|
||||||
|
op.drop_index("ix_notifications_type", table_name="notifications")
|
||||||
|
op.drop_index("ix_notifications_user_id", table_name="notifications")
|
||||||
|
op.drop_table("notifications")
|
||||||
|
|
@ -32,6 +32,7 @@ from app.routes import (
|
||||||
feed,
|
feed,
|
||||||
health,
|
health,
|
||||||
me,
|
me,
|
||||||
|
notifications,
|
||||||
playlists,
|
playlists,
|
||||||
quota,
|
quota,
|
||||||
scheduler as scheduler_routes,
|
scheduler as scheduler_routes,
|
||||||
|
|
@ -77,6 +78,7 @@ app.include_router(sync.router)
|
||||||
app.include_router(tags.router)
|
app.include_router(tags.router)
|
||||||
app.include_router(feed.router)
|
app.include_router(feed.router)
|
||||||
app.include_router(me.router)
|
app.include_router(me.router)
|
||||||
|
app.include_router(notifications.router)
|
||||||
app.include_router(channels.router)
|
app.include_router(channels.router)
|
||||||
app.include_router(playlists.router)
|
app.include_router(playlists.router)
|
||||||
app.include_router(admin.router)
|
app.include_router(admin.router)
|
||||||
|
|
|
||||||
|
|
@ -203,6 +203,22 @@ class Video(Base):
|
||||||
String(16), default="none", server_default="none", index=True
|
String(16), default="none", server_default="none", index=True
|
||||||
)
|
)
|
||||||
enriched_at: Mapped[datetime | None] = mapped_column(DateTime(timezone=True))
|
enriched_at: Mapped[datetime | None] = mapped_column(DateTime(timezone=True))
|
||||||
|
# videos.list?part=status fields (populated during enrichment); used by the
|
||||||
|
# maintenance/validation job to decide whether a video is playable anywhere.
|
||||||
|
embeddable: Mapped[bool | None] = mapped_column(Boolean)
|
||||||
|
privacy_status: Mapped[str | None] = mapped_column(String(16)) # public|unlisted|private
|
||||||
|
upload_status: Mapped[str | None] = mapped_column(String(16)) # processed|failed|rejected|deleted
|
||||||
|
# Maintenance/validation lifecycle. last_checked_at drives the rolling re-validation
|
||||||
|
# (least-recently-checked first); unavailable_since marks a video as currently
|
||||||
|
# unplayable (hidden from the feed immediately, hard-deleted after a grace period);
|
||||||
|
# unavailable_reason records why (deleted|private|paywalled|abandoned|stuck_live).
|
||||||
|
last_checked_at: Mapped[datetime | None] = mapped_column(
|
||||||
|
DateTime(timezone=True), index=True
|
||||||
|
)
|
||||||
|
unavailable_since: Mapped[datetime | None] = mapped_column(
|
||||||
|
DateTime(timezone=True), index=True
|
||||||
|
)
|
||||||
|
unavailable_reason: Mapped[str | None] = mapped_column(String(24))
|
||||||
created_at: Mapped[datetime] = mapped_column(
|
created_at: Mapped[datetime] = mapped_column(
|
||||||
DateTime(timezone=True), server_default=func.now()
|
DateTime(timezone=True), server_default=func.now()
|
||||||
)
|
)
|
||||||
|
|
@ -399,3 +415,36 @@ class PlaylistItem(Base):
|
||||||
)
|
)
|
||||||
|
|
||||||
playlist: Mapped["Playlist"] = relationship(back_populates="items")
|
playlist: Mapped["Playlist"] = relationship(back_populates="items")
|
||||||
|
|
||||||
|
|
||||||
|
class Notification(Base):
|
||||||
|
"""A durable, server-backed per-user notification (the inbox center, phase 1).
|
||||||
|
|
||||||
|
Distinct from the client-side transient "bell" (toasts kept only in localStorage):
|
||||||
|
these survive reloads and devices, and are produced server-side (first producer = the
|
||||||
|
maintenance/validation job's batched "N saved/playlisted videos removed" notice). The
|
||||||
|
column is named `data` (not `metadata`, which SQLAlchemy's declarative Base reserves):
|
||||||
|
a free-form JSON payload (e.g. the list of removed videos) the UI can render. `read`
|
||||||
|
and `dismissed` are separate: read clears the unread badge but keeps the row in the
|
||||||
|
inbox; dismissed removes it from the default view. Unread rows never auto-expire; read
|
||||||
|
ones are trimmed past a soft per-user cap (see app.notifications.trim_read)."""
|
||||||
|
|
||||||
|
__tablename__ = "notifications"
|
||||||
|
|
||||||
|
id: Mapped[int] = mapped_column(primary_key=True)
|
||||||
|
user_id: Mapped[int] = mapped_column(
|
||||||
|
ForeignKey("users.id", ondelete="CASCADE"), index=True
|
||||||
|
)
|
||||||
|
type: Mapped[str] = mapped_column(String(32), index=True) # e.g. "maintenance"
|
||||||
|
title: Mapped[str] = mapped_column(String(255))
|
||||||
|
body: Mapped[str | None] = mapped_column(Text)
|
||||||
|
data: Mapped[dict | None] = mapped_column(JSON)
|
||||||
|
read: Mapped[bool] = mapped_column(
|
||||||
|
Boolean, default=False, server_default="false", index=True
|
||||||
|
)
|
||||||
|
dismissed: Mapped[bool] = mapped_column(
|
||||||
|
Boolean, default=False, server_default="false"
|
||||||
|
)
|
||||||
|
created_at: Mapped[datetime] = mapped_column(
|
||||||
|
DateTime(timezone=True), server_default=func.now(), index=True
|
||||||
|
)
|
||||||
|
|
|
||||||
58
backend/app/notifications.py
Normal file
58
backend/app/notifications.py
Normal file
|
|
@ -0,0 +1,58 @@
|
||||||
|
"""Helpers for the durable, server-backed per-user notification inbox (phase 1).
|
||||||
|
|
||||||
|
The inbox COEXISTS with the client-side transient "bell" (localStorage toasts): these rows
|
||||||
|
survive reloads/devices and are produced server-side. First producer is the maintenance
|
||||||
|
job. Unread rows never auto-expire; read rows are trimmed past a soft per-user cap so the
|
||||||
|
table can't grow without bound for a heavy user.
|
||||||
|
"""
|
||||||
|
from sqlalchemy import select
|
||||||
|
from sqlalchemy.orm import Session
|
||||||
|
|
||||||
|
from app.models import Notification
|
||||||
|
|
||||||
|
# Keep at most this many READ notifications per user; older read ones are pruned. Unread
|
||||||
|
# rows are never pruned (the user hasn't seen them yet).
|
||||||
|
READ_SOFT_CAP = 200
|
||||||
|
|
||||||
|
|
||||||
|
def create_notification(
|
||||||
|
db: Session,
|
||||||
|
user_id: int,
|
||||||
|
type: str,
|
||||||
|
title: str,
|
||||||
|
body: str | None = None,
|
||||||
|
data: dict | None = None,
|
||||||
|
*,
|
||||||
|
commit: bool = True,
|
||||||
|
) -> Notification:
|
||||||
|
"""Insert one notification for a user. Callers that batch many inserts in a single
|
||||||
|
transaction can pass commit=False and commit once themselves."""
|
||||||
|
notif = Notification(
|
||||||
|
user_id=user_id, type=type, title=title, body=body, data=data
|
||||||
|
)
|
||||||
|
db.add(notif)
|
||||||
|
if commit:
|
||||||
|
db.commit()
|
||||||
|
db.refresh(notif)
|
||||||
|
return notif
|
||||||
|
|
||||||
|
|
||||||
|
def trim_read(db: Session, user_id: int, cap: int = READ_SOFT_CAP) -> int:
|
||||||
|
"""Delete a user's oldest READ notifications beyond `cap`. Returns the number removed.
|
||||||
|
Idempotent and cheap; safe to call after marking things read."""
|
||||||
|
ids = (
|
||||||
|
db.execute(
|
||||||
|
select(Notification.id)
|
||||||
|
.where(Notification.user_id == user_id, Notification.read.is_(True))
|
||||||
|
.order_by(Notification.created_at.desc())
|
||||||
|
.offset(cap)
|
||||||
|
)
|
||||||
|
.scalars()
|
||||||
|
.all()
|
||||||
|
)
|
||||||
|
if not ids:
|
||||||
|
return 0
|
||||||
|
for nid in ids:
|
||||||
|
db.delete(db.get(Notification, nid))
|
||||||
|
db.commit()
|
||||||
|
return len(ids)
|
||||||
136
backend/app/routes/notifications.py
Normal file
136
backend/app/routes/notifications.py
Normal file
|
|
@ -0,0 +1,136 @@
|
||||||
|
"""Per-user notification inbox (phase 1): the durable, server-backed center that coexists
|
||||||
|
with the client-side transient bell. Read-only listing plus read/dismiss/clear mutations.
|
||||||
|
|
||||||
|
All endpoints use `current_user` (not `require_human`): notifications and their read/dismiss
|
||||||
|
state are personal UI state with no quota or YouTube involvement, so the shared demo account
|
||||||
|
manages its own inbox normally."""
|
||||||
|
from fastapi import APIRouter, Depends, HTTPException, Query
|
||||||
|
from sqlalchemy import func, select, update
|
||||||
|
from sqlalchemy.orm import Session
|
||||||
|
|
||||||
|
from app.auth import current_user
|
||||||
|
from app.db import get_db
|
||||||
|
from app.models import Notification, User
|
||||||
|
from app.notifications import trim_read
|
||||||
|
|
||||||
|
router = APIRouter(prefix="/api/me/notifications", tags=["notifications"])
|
||||||
|
|
||||||
|
|
||||||
|
def _serialize(n: Notification) -> dict:
|
||||||
|
return {
|
||||||
|
"id": n.id,
|
||||||
|
"type": n.type,
|
||||||
|
"title": n.title,
|
||||||
|
"body": n.body,
|
||||||
|
"data": n.data,
|
||||||
|
"read": n.read,
|
||||||
|
"dismissed": n.dismissed,
|
||||||
|
"created_at": n.created_at.isoformat() if n.created_at else None,
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
@router.get("")
|
||||||
|
def list_notifications(
|
||||||
|
include_dismissed: bool = False,
|
||||||
|
limit: int = Query(default=100, le=500),
|
||||||
|
offset: int = 0,
|
||||||
|
user: User = Depends(current_user),
|
||||||
|
db: Session = Depends(get_db),
|
||||||
|
) -> dict:
|
||||||
|
where = [Notification.user_id == user.id]
|
||||||
|
if not include_dismissed:
|
||||||
|
where.append(Notification.dismissed.is_(False))
|
||||||
|
total = db.scalar(
|
||||||
|
select(func.count()).select_from(Notification).where(*where)
|
||||||
|
) or 0
|
||||||
|
rows = (
|
||||||
|
db.execute(
|
||||||
|
select(Notification)
|
||||||
|
.where(*where)
|
||||||
|
.order_by(Notification.created_at.desc())
|
||||||
|
.offset(offset)
|
||||||
|
.limit(limit)
|
||||||
|
)
|
||||||
|
.scalars()
|
||||||
|
.all()
|
||||||
|
)
|
||||||
|
return {"items": [_serialize(n) for n in rows], "total": total}
|
||||||
|
|
||||||
|
|
||||||
|
@router.get("/unread_count")
|
||||||
|
def unread_count(
|
||||||
|
user: User = Depends(current_user), db: Session = Depends(get_db)
|
||||||
|
) -> dict:
|
||||||
|
"""Cheap poll target for the nav badge (uses the (user_id, read) index)."""
|
||||||
|
count = db.scalar(
|
||||||
|
select(func.count())
|
||||||
|
.select_from(Notification)
|
||||||
|
.where(
|
||||||
|
Notification.user_id == user.id,
|
||||||
|
Notification.read.is_(False),
|
||||||
|
Notification.dismissed.is_(False),
|
||||||
|
)
|
||||||
|
)
|
||||||
|
return {"count": count or 0}
|
||||||
|
|
||||||
|
|
||||||
|
def _owned(db: Session, notification_id: int, user: User) -> Notification:
|
||||||
|
n = db.get(Notification, notification_id)
|
||||||
|
if n is None or n.user_id != user.id:
|
||||||
|
raise HTTPException(status_code=404, detail="Notification not found")
|
||||||
|
return n
|
||||||
|
|
||||||
|
|
||||||
|
@router.post("/{notification_id}/read")
|
||||||
|
def mark_read(
|
||||||
|
notification_id: int,
|
||||||
|
user: User = Depends(current_user),
|
||||||
|
db: Session = Depends(get_db),
|
||||||
|
) -> dict:
|
||||||
|
n = _owned(db, notification_id, user)
|
||||||
|
n.read = True
|
||||||
|
db.commit()
|
||||||
|
trim_read(db, user.id)
|
||||||
|
return {"ok": True}
|
||||||
|
|
||||||
|
|
||||||
|
@router.post("/read_all")
|
||||||
|
def mark_all_read(
|
||||||
|
user: User = Depends(current_user), db: Session = Depends(get_db)
|
||||||
|
) -> dict:
|
||||||
|
db.execute(
|
||||||
|
update(Notification)
|
||||||
|
.where(Notification.user_id == user.id, Notification.read.is_(False))
|
||||||
|
.values(read=True)
|
||||||
|
)
|
||||||
|
db.commit()
|
||||||
|
trim_read(db, user.id)
|
||||||
|
return {"ok": True}
|
||||||
|
|
||||||
|
|
||||||
|
@router.post("/{notification_id}/dismiss")
|
||||||
|
def dismiss(
|
||||||
|
notification_id: int,
|
||||||
|
user: User = Depends(current_user),
|
||||||
|
db: Session = Depends(get_db),
|
||||||
|
) -> dict:
|
||||||
|
n = _owned(db, notification_id, user)
|
||||||
|
# Dismiss implies read (it no longer counts toward the unread badge).
|
||||||
|
n.dismissed = True
|
||||||
|
n.read = True
|
||||||
|
db.commit()
|
||||||
|
return {"ok": True}
|
||||||
|
|
||||||
|
|
||||||
|
@router.post("/clear")
|
||||||
|
def clear_all(
|
||||||
|
user: User = Depends(current_user), db: Session = Depends(get_db)
|
||||||
|
) -> dict:
|
||||||
|
"""Dismiss every (still-visible) notification for this user."""
|
||||||
|
db.execute(
|
||||||
|
update(Notification)
|
||||||
|
.where(Notification.user_id == user.id, Notification.dismissed.is_(False))
|
||||||
|
.values(dismissed=True, read=True)
|
||||||
|
)
|
||||||
|
db.commit()
|
||||||
|
return {"ok": True}
|
||||||
|
|
@ -10,7 +10,7 @@ import {
|
||||||
saveLocalTheme,
|
saveLocalTheme,
|
||||||
type ThemePrefs,
|
type ThemePrefs,
|
||||||
} from "./lib/theme";
|
} from "./lib/theme";
|
||||||
import { hasFilterParams, paramsToFilters, readPage, stripUrlParams, type Page } from "./lib/urlState";
|
import { hasFilterParams, isPage, paramsToFilters, readPage, stripUrlParams, type Page } from "./lib/urlState";
|
||||||
import {
|
import {
|
||||||
loadLayout,
|
loadLayout,
|
||||||
normalizeLayout,
|
normalizeLayout,
|
||||||
|
|
@ -29,6 +29,7 @@ import Playlists from "./components/Playlists";
|
||||||
import Stats from "./components/Stats";
|
import Stats from "./components/Stats";
|
||||||
import Scheduler from "./components/Scheduler";
|
import Scheduler from "./components/Scheduler";
|
||||||
import SettingsPanel from "./components/SettingsPanel";
|
import SettingsPanel from "./components/SettingsPanel";
|
||||||
|
import NotificationsPanel from "./components/NotificationsPanel";
|
||||||
import OnboardingWizard from "./components/OnboardingWizard";
|
import OnboardingWizard from "./components/OnboardingWizard";
|
||||||
import { shouldAutoOpenOnboarding } from "./lib/onboarding";
|
import { shouldAutoOpenOnboarding } from "./lib/onboarding";
|
||||||
import Toaster from "./components/Toaster";
|
import Toaster from "./components/Toaster";
|
||||||
|
|
@ -60,15 +61,7 @@ function loadInitialPage(): Page {
|
||||||
const params = new URLSearchParams(window.location.search);
|
const params = new URLSearchParams(window.location.search);
|
||||||
if (params.has("page")) return readPage();
|
if (params.has("page")) return readPage();
|
||||||
const stored = localStorage.getItem(PAGE_KEY);
|
const stored = localStorage.getItem(PAGE_KEY);
|
||||||
if (
|
return isPage(stored) ? stored : "feed";
|
||||||
stored === "channels" ||
|
|
||||||
stored === "stats" ||
|
|
||||||
stored === "playlists" ||
|
|
||||||
stored === "settings" ||
|
|
||||||
stored === "scheduler"
|
|
||||||
)
|
|
||||||
return stored;
|
|
||||||
return "feed";
|
|
||||||
}
|
}
|
||||||
|
|
||||||
function loadStoredFilters(): FeedFilters {
|
function loadStoredFilters(): FeedFilters {
|
||||||
|
|
@ -311,6 +304,8 @@ export default function App() {
|
||||||
<Scheduler />
|
<Scheduler />
|
||||||
) : page === "playlists" ? (
|
) : page === "playlists" ? (
|
||||||
<Playlists canWrite={meQuery.data!.can_write} />
|
<Playlists canWrite={meQuery.data!.can_write} />
|
||||||
|
) : page === "notifications" ? (
|
||||||
|
<NotificationsPanel />
|
||||||
) : page === "settings" ? (
|
) : page === "settings" ? (
|
||||||
<SettingsPanel
|
<SettingsPanel
|
||||||
me={meQuery.data!}
|
me={meQuery.data!}
|
||||||
|
|
|
||||||
|
|
@ -5,6 +5,7 @@ import { useQuery } from "@tanstack/react-query";
|
||||||
import {
|
import {
|
||||||
Activity,
|
Activity,
|
||||||
BarChart3,
|
BarChart3,
|
||||||
|
Bell,
|
||||||
ChevronLeft,
|
ChevronLeft,
|
||||||
ChevronRight,
|
ChevronRight,
|
||||||
Home,
|
Home,
|
||||||
|
|
@ -17,6 +18,7 @@ import {
|
||||||
UserPlus,
|
UserPlus,
|
||||||
} from "lucide-react";
|
} from "lucide-react";
|
||||||
import { api, type FeedFilters, type Me } from "../lib/api";
|
import { api, type FeedFilters, type Me } from "../lib/api";
|
||||||
|
import { useLiveQuery } from "../lib/useLiveQuery";
|
||||||
import type { Page } from "../lib/urlState";
|
import type { Page } from "../lib/urlState";
|
||||||
import { type LangCode } from "../i18n";
|
import { type LangCode } from "../i18n";
|
||||||
import AvatarImg from "./Avatar";
|
import AvatarImg from "./Avatar";
|
||||||
|
|
@ -116,12 +118,22 @@ export default function NavSidebar({
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
type NavItem = { page: Page; icon: typeof Home; label: string };
|
// Durable, server-backed unread count for the inbox badge. Polled live (pauses when the
|
||||||
|
// tab is hidden); coexists with the client-side transient bell at the bottom of the rail.
|
||||||
|
const unreadQuery = useLiveQuery(
|
||||||
|
["notif-unread"],
|
||||||
|
api.notificationUnreadCount,
|
||||||
|
{ intervalMs: 30000 }
|
||||||
|
);
|
||||||
|
const unread = unreadQuery.data?.count ?? 0;
|
||||||
|
|
||||||
|
type NavItem = { page: Page; icon: typeof Home; label: string; badge?: number };
|
||||||
// User-facing content modules vs. admin/system modules, separated by a divider in the rail.
|
// User-facing content modules vs. admin/system modules, separated by a divider in the rail.
|
||||||
const userItems: NavItem[] = [
|
const userItems: NavItem[] = [
|
||||||
{ page: "feed", icon: Home, label: t("header.account.feed") },
|
{ page: "feed", icon: Home, label: t("header.account.feed") },
|
||||||
{ page: "channels", icon: Tv, label: t("header.account.channels") },
|
{ page: "channels", icon: Tv, label: t("header.account.channels") },
|
||||||
{ page: "playlists", icon: ListVideo, label: t("header.account.playlists") },
|
{ page: "playlists", icon: ListVideo, label: t("header.account.playlists") },
|
||||||
|
{ page: "notifications", icon: Bell, label: t("inbox.navLabel"), badge: unread },
|
||||||
];
|
];
|
||||||
const systemItems: NavItem[] =
|
const systemItems: NavItem[] =
|
||||||
me.role === "admin"
|
me.role === "admin"
|
||||||
|
|
@ -135,18 +147,29 @@ export default function NavSidebar({
|
||||||
"w-full flex items-center gap-3 rounded-lg px-2.5 py-2 text-sm transition";
|
"w-full flex items-center gap-3 rounded-lg px-2.5 py-2 text-sm transition";
|
||||||
const name = me.display_name ?? me.email.split("@")[0];
|
const name = me.display_name ?? me.email.split("@")[0];
|
||||||
|
|
||||||
const renderItem = ({ page: p, icon: Icon, label }: NavItem) => (
|
const renderItem = ({ page: p, icon: Icon, label, badge }: NavItem) => (
|
||||||
<button
|
<button
|
||||||
key={p}
|
key={p}
|
||||||
onClick={() => setPage(p)}
|
onClick={() => setPage(p)}
|
||||||
title={collapsed ? label : undefined}
|
title={collapsed ? label : undefined}
|
||||||
aria-current={page === p ? "page" : undefined}
|
aria-current={page === p ? "page" : undefined}
|
||||||
className={`${rowBase} ${collapsed ? "justify-center px-0" : ""} ${
|
className={`${rowBase} ${collapsed ? "justify-center px-0" : ""} relative ${
|
||||||
page === p ? "bg-accent text-accent-fg" : "text-muted hover:text-fg hover:bg-card"
|
page === p ? "bg-accent text-accent-fg" : "text-muted hover:text-fg hover:bg-card"
|
||||||
}`}
|
}`}
|
||||||
>
|
>
|
||||||
<Icon className="w-[18px] h-[18px] shrink-0" />
|
<span className="relative shrink-0">
|
||||||
|
<Icon className="w-[18px] h-[18px]" />
|
||||||
|
{/* Collapsed rail: a small dot on the icon; expanded: a numeric pill at the row end. */}
|
||||||
|
{!!badge && badge > 0 && collapsed && (
|
||||||
|
<span className="absolute -top-1 -right-1 w-2 h-2 rounded-full bg-accent ring-2 ring-bg" />
|
||||||
|
)}
|
||||||
|
</span>
|
||||||
{!collapsed && <span className="truncate">{label}</span>}
|
{!collapsed && <span className="truncate">{label}</span>}
|
||||||
|
{!!badge && badge > 0 && !collapsed && (
|
||||||
|
<span className="ml-auto min-w-[18px] h-[18px] px-1 rounded-full bg-accent text-accent-fg text-[11px] font-semibold inline-flex items-center justify-center tabular-nums">
|
||||||
|
{badge > 99 ? "99+" : badge}
|
||||||
|
</span>
|
||||||
|
)}
|
||||||
</button>
|
</button>
|
||||||
);
|
);
|
||||||
|
|
||||||
|
|
|
||||||
176
frontend/src/components/NotificationsPanel.tsx
Normal file
176
frontend/src/components/NotificationsPanel.tsx
Normal file
|
|
@ -0,0 +1,176 @@
|
||||||
|
import { useTranslation } from "react-i18next";
|
||||||
|
import { useMutation, useQueryClient } from "@tanstack/react-query";
|
||||||
|
import { Bell, Check, CheckCheck, Trash2, X } from "lucide-react";
|
||||||
|
import { api, type AppNotification } from "../lib/api";
|
||||||
|
import { useLiveQuery } from "../lib/useLiveQuery";
|
||||||
|
|
||||||
|
// The durable, server-backed notification center (inbox phase 1). It COEXISTS with the
|
||||||
|
// client-side transient bell in the nav rail: the bell is a quick peek at recent toasts kept
|
||||||
|
// in localStorage, this is the full, filterable, clearable history synced from the server.
|
||||||
|
const POLL_MS = 8000;
|
||||||
|
|
||||||
|
function relativeTime(iso: string | null, t: (k: string, o?: any) => string): string {
|
||||||
|
if (!iso) return "";
|
||||||
|
const then = new Date(iso).getTime();
|
||||||
|
const secs = Math.max(0, Math.round((Date.now() - then) / 1000));
|
||||||
|
// Reuse the bell's relative-time strings (notifications.time.*) — same format, no dupes.
|
||||||
|
if (secs < 60) return t("notifications.time.justNow");
|
||||||
|
const mins = Math.round(secs / 60);
|
||||||
|
if (mins < 60) return t("notifications.time.minutes", { count: mins });
|
||||||
|
const hours = Math.round(mins / 60);
|
||||||
|
if (hours < 24) return t("notifications.time.hours", { count: hours });
|
||||||
|
const days = Math.round(hours / 24);
|
||||||
|
return t("notifications.time.days", { count: days });
|
||||||
|
}
|
||||||
|
|
||||||
|
export default function NotificationsPanel() {
|
||||||
|
const { t } = useTranslation();
|
||||||
|
const qc = useQueryClient();
|
||||||
|
const q = useLiveQuery<{ items: AppNotification[]; total: number }>(
|
||||||
|
["notifications"],
|
||||||
|
() => api.notifications(),
|
||||||
|
{ intervalMs: POLL_MS }
|
||||||
|
);
|
||||||
|
const items = q.data?.items ?? [];
|
||||||
|
|
||||||
|
// Any mutation refreshes both the list and the nav badge's unread count.
|
||||||
|
const refresh = () => {
|
||||||
|
qc.invalidateQueries({ queryKey: ["notifications"] });
|
||||||
|
qc.invalidateQueries({ queryKey: ["notif-unread"] });
|
||||||
|
};
|
||||||
|
const readMut = useMutation({ mutationFn: api.markNotificationRead, onSuccess: refresh });
|
||||||
|
const readAllMut = useMutation({ mutationFn: api.markAllNotificationsRead, onSuccess: refresh });
|
||||||
|
const dismissMut = useMutation({ mutationFn: api.dismissNotification, onSuccess: refresh });
|
||||||
|
const clearMut = useMutation({ mutationFn: api.clearNotifications, onSuccess: refresh });
|
||||||
|
|
||||||
|
const hasUnread = items.some((n) => !n.read);
|
||||||
|
|
||||||
|
return (
|
||||||
|
<div className="p-4 max-w-3xl w-full mx-auto space-y-4">
|
||||||
|
<div className="glass rounded-2xl p-4 flex items-center gap-3">
|
||||||
|
<Bell className="w-5 h-5 text-accent shrink-0" />
|
||||||
|
<div className="flex-1 min-w-0">
|
||||||
|
<div className="font-semibold">{t("inbox.title")}</div>
|
||||||
|
<div className="text-xs text-muted">{t("inbox.subtitle")}</div>
|
||||||
|
</div>
|
||||||
|
{items.length > 0 && (
|
||||||
|
<div className="flex items-center gap-2">
|
||||||
|
<button
|
||||||
|
onClick={() => readAllMut.mutate()}
|
||||||
|
disabled={!hasUnread || readAllMut.isPending}
|
||||||
|
className="inline-flex items-center gap-1.5 text-xs px-2.5 py-1.5 rounded-lg text-muted hover:text-fg hover:bg-card transition disabled:opacity-40"
|
||||||
|
>
|
||||||
|
<CheckCheck className="w-4 h-4" />
|
||||||
|
{t("inbox.markAllRead")}
|
||||||
|
</button>
|
||||||
|
<button
|
||||||
|
onClick={() => clearMut.mutate()}
|
||||||
|
disabled={clearMut.isPending}
|
||||||
|
className="inline-flex items-center gap-1.5 text-xs px-2.5 py-1.5 rounded-lg text-muted hover:text-fg hover:bg-card transition disabled:opacity-40"
|
||||||
|
>
|
||||||
|
<Trash2 className="w-4 h-4" />
|
||||||
|
{t("inbox.clearAll")}
|
||||||
|
</button>
|
||||||
|
</div>
|
||||||
|
)}
|
||||||
|
</div>
|
||||||
|
|
||||||
|
{q.isLoading && !q.data ? (
|
||||||
|
<div className="p-8 text-muted text-center">{t("common.loading")}</div>
|
||||||
|
) : items.length === 0 ? (
|
||||||
|
<div className="glass rounded-2xl p-10 text-center text-muted">
|
||||||
|
<Bell className="w-8 h-8 mx-auto mb-3 opacity-40" />
|
||||||
|
{t("inbox.empty")}
|
||||||
|
</div>
|
||||||
|
) : (
|
||||||
|
<div className="flex flex-col gap-2">
|
||||||
|
{items.map((n) => (
|
||||||
|
<NotificationRow
|
||||||
|
key={n.id}
|
||||||
|
n={n}
|
||||||
|
t={t}
|
||||||
|
onRead={() => readMut.mutate(n.id)}
|
||||||
|
onDismiss={() => dismissMut.mutate(n.id)}
|
||||||
|
/>
|
||||||
|
))}
|
||||||
|
</div>
|
||||||
|
)}
|
||||||
|
</div>
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
function NotificationRow({
|
||||||
|
n,
|
||||||
|
t,
|
||||||
|
onRead,
|
||||||
|
onDismiss,
|
||||||
|
}: {
|
||||||
|
n: AppNotification;
|
||||||
|
t: (k: string, o?: any) => string;
|
||||||
|
onRead: () => void;
|
||||||
|
onDismiss: () => void;
|
||||||
|
}) {
|
||||||
|
// The maintenance producer ships the affected videos in `data.videos`; list a few titles.
|
||||||
|
const videos: { id: string; title: string | null }[] = Array.isArray(n.data?.videos)
|
||||||
|
? n.data!.videos
|
||||||
|
: [];
|
||||||
|
// Known notification types are rendered from i18n (so they're trilingual) using the typed
|
||||||
|
// payload; the server-stored title/body are an English fallback for any unknown type.
|
||||||
|
const isMaintenance = n.type === "maintenance" && typeof n.data?.count === "number";
|
||||||
|
const title = isMaintenance ? t("inbox.maintenance.title") : n.title;
|
||||||
|
const body = isMaintenance
|
||||||
|
? t("inbox.maintenance.body", { count: n.data!.count })
|
||||||
|
: n.body;
|
||||||
|
return (
|
||||||
|
<div
|
||||||
|
className={`glass rounded-xl p-3.5 flex items-start gap-3 transition ${
|
||||||
|
n.read ? "opacity-70" : ""
|
||||||
|
}`}
|
||||||
|
>
|
||||||
|
{!n.read && <span className="mt-1.5 w-2 h-2 rounded-full bg-accent shrink-0" aria-hidden />}
|
||||||
|
<div className="flex-1 min-w-0">
|
||||||
|
<div className="flex items-baseline gap-2">
|
||||||
|
<span className="font-medium text-sm">{title}</span>
|
||||||
|
<span className="text-[11px] text-muted shrink-0">
|
||||||
|
{relativeTime(n.created_at, t)}
|
||||||
|
</span>
|
||||||
|
</div>
|
||||||
|
{body && <div className="text-sm text-muted mt-0.5">{body}</div>}
|
||||||
|
{videos.length > 0 && (
|
||||||
|
<ul className="mt-2 text-xs text-muted list-disc pl-4 space-y-0.5">
|
||||||
|
{videos.slice(0, 8).map((v) => (
|
||||||
|
<li key={v.id} className="truncate">
|
||||||
|
{v.title || v.id}
|
||||||
|
</li>
|
||||||
|
))}
|
||||||
|
{videos.length > 8 && (
|
||||||
|
<li className="list-none text-muted/70">
|
||||||
|
{t("inbox.andMore", { count: videos.length - 8 })}
|
||||||
|
</li>
|
||||||
|
)}
|
||||||
|
</ul>
|
||||||
|
)}
|
||||||
|
</div>
|
||||||
|
<div className="flex items-center gap-1 shrink-0">
|
||||||
|
{!n.read && (
|
||||||
|
<button
|
||||||
|
onClick={onRead}
|
||||||
|
title={t("inbox.markRead")}
|
||||||
|
aria-label={t("inbox.markRead")}
|
||||||
|
className="p-1.5 rounded-lg text-muted hover:text-fg hover:bg-card transition"
|
||||||
|
>
|
||||||
|
<Check className="w-4 h-4" />
|
||||||
|
</button>
|
||||||
|
)}
|
||||||
|
<button
|
||||||
|
onClick={onDismiss}
|
||||||
|
title={t("inbox.dismiss")}
|
||||||
|
aria-label={t("inbox.dismiss")}
|
||||||
|
className="p-1.5 rounded-lg text-muted hover:text-fg hover:bg-card transition"
|
||||||
|
>
|
||||||
|
<X className="w-4 h-4" />
|
||||||
|
</button>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
);
|
||||||
|
}
|
||||||
16
frontend/src/i18n/locales/de/inbox.json
Normal file
16
frontend/src/i18n/locales/de/inbox.json
Normal file
|
|
@ -0,0 +1,16 @@
|
||||||
|
{
|
||||||
|
"navLabel": "Benachrichtigungen",
|
||||||
|
"title": "Benachrichtigungen",
|
||||||
|
"subtitle": "Neuigkeiten aus deiner Bibliothek und vom System.",
|
||||||
|
"empty": "Alles erledigt.",
|
||||||
|
"markAllRead": "Alle als gelesen markieren",
|
||||||
|
"clearAll": "Alle löschen",
|
||||||
|
"markRead": "Als gelesen markieren",
|
||||||
|
"dismiss": "Verwerfen",
|
||||||
|
"andMore": "und {{count}} weitere",
|
||||||
|
"maintenance": {
|
||||||
|
"title": "Videos entfernt",
|
||||||
|
"body_one": "{{count}} gespeichertes oder in einer Playlist befindliches Video wurde entfernt, weil es auf YouTube nicht mehr verfügbar ist.",
|
||||||
|
"body_other": "{{count}} gespeicherte oder in Playlists befindliche Videos wurden entfernt, weil sie auf YouTube nicht mehr verfügbar sind."
|
||||||
|
}
|
||||||
|
}
|
||||||
16
frontend/src/i18n/locales/en/inbox.json
Normal file
16
frontend/src/i18n/locales/en/inbox.json
Normal file
|
|
@ -0,0 +1,16 @@
|
||||||
|
{
|
||||||
|
"navLabel": "Notifications",
|
||||||
|
"title": "Notifications",
|
||||||
|
"subtitle": "Updates from your library and the system.",
|
||||||
|
"empty": "You're all caught up.",
|
||||||
|
"markAllRead": "Mark all read",
|
||||||
|
"clearAll": "Clear all",
|
||||||
|
"markRead": "Mark read",
|
||||||
|
"dismiss": "Dismiss",
|
||||||
|
"andMore": "and {{count}} more",
|
||||||
|
"maintenance": {
|
||||||
|
"title": "Videos removed",
|
||||||
|
"body_one": "{{count}} saved or playlisted video was removed because it's no longer available on YouTube.",
|
||||||
|
"body_other": "{{count}} saved or playlisted videos were removed because they're no longer available on YouTube."
|
||||||
|
}
|
||||||
|
}
|
||||||
16
frontend/src/i18n/locales/hu/inbox.json
Normal file
16
frontend/src/i18n/locales/hu/inbox.json
Normal file
|
|
@ -0,0 +1,16 @@
|
||||||
|
{
|
||||||
|
"navLabel": "Értesítések",
|
||||||
|
"title": "Értesítések",
|
||||||
|
"subtitle": "Frissítések a könyvtáradból és a rendszertől.",
|
||||||
|
"empty": "Nincs új értesítés.",
|
||||||
|
"markAllRead": "Összes olvasott",
|
||||||
|
"clearAll": "Összes törlése",
|
||||||
|
"markRead": "Olvasottnak jelöl",
|
||||||
|
"dismiss": "Elvetés",
|
||||||
|
"andMore": "és még {{count}}",
|
||||||
|
"maintenance": {
|
||||||
|
"title": "Videók eltávolítva",
|
||||||
|
"body_one": "{{count}} mentett vagy lejátszási listás videó eltávolítva, mert már nem elérhető a YouTube-on.",
|
||||||
|
"body_other": "{{count}} mentett vagy lejátszási listás videó eltávolítva, mert már nem elérhetők a YouTube-on."
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
@ -356,6 +356,19 @@ export interface Account {
|
||||||
active: boolean;
|
active: boolean;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// A durable, server-backed notification (the inbox center). Distinct from the client-side
|
||||||
|
// transient toast/bell, which lives only in localStorage.
|
||||||
|
export interface AppNotification {
|
||||||
|
id: number;
|
||||||
|
type: string;
|
||||||
|
title: string;
|
||||||
|
body: string | null;
|
||||||
|
data: Record<string, any> | null;
|
||||||
|
read: boolean;
|
||||||
|
dismissed: boolean;
|
||||||
|
created_at: string | null;
|
||||||
|
}
|
||||||
|
|
||||||
export const api = {
|
export const api = {
|
||||||
me: (): Promise<Me> => req("/api/me"),
|
me: (): Promise<Me> => req("/api/me"),
|
||||||
accounts: (): Promise<Account[]> => req("/api/me/accounts"),
|
accounts: (): Promise<Account[]> => req("/api/me/accounts"),
|
||||||
|
|
@ -478,6 +491,19 @@ export const api = {
|
||||||
addInvite: (email: string) =>
|
addInvite: (email: string) =>
|
||||||
req("/api/admin/invites", { method: "POST", body: JSON.stringify({ email }) }),
|
req("/api/admin/invites", { method: "POST", body: JSON.stringify({ email }) }),
|
||||||
|
|
||||||
|
// --- notification inbox (durable, server-backed) ---
|
||||||
|
notifications: (includeDismissed = false): Promise<{ items: AppNotification[]; total: number }> =>
|
||||||
|
req(`/api/me/notifications?include_dismissed=${includeDismissed}`),
|
||||||
|
notificationUnreadCount: (): Promise<{ count: number }> =>
|
||||||
|
req("/api/me/notifications/unread_count"),
|
||||||
|
markNotificationRead: (id: number) =>
|
||||||
|
req(`/api/me/notifications/${id}/read`, { method: "POST" }),
|
||||||
|
markAllNotificationsRead: () =>
|
||||||
|
req("/api/me/notifications/read_all", { method: "POST" }),
|
||||||
|
dismissNotification: (id: number) =>
|
||||||
|
req(`/api/me/notifications/${id}/dismiss`, { method: "POST" }),
|
||||||
|
clearNotifications: () => req("/api/me/notifications/clear", { method: "POST" }),
|
||||||
|
|
||||||
// --- user tags ---
|
// --- user tags ---
|
||||||
createTag: (t: { name: string; color?: string; category?: string }) =>
|
createTag: (t: { name: string; color?: string; category?: string }) =>
|
||||||
req("/api/tags", { method: "POST", body: JSON.stringify(t) }),
|
req("/api/tags", { method: "POST", body: JSON.stringify(t) }),
|
||||||
|
|
|
||||||
|
|
@ -78,17 +78,29 @@ export function hasFilterParams(params: URLSearchParams): boolean {
|
||||||
return KEYS.some((k) => params.has(k));
|
return KEYS.some((k) => params.has(k));
|
||||||
}
|
}
|
||||||
|
|
||||||
export type Page = "feed" | "channels" | "stats" | "playlists" | "settings" | "scheduler";
|
// The single source of truth for valid page ids; `Page` and the runtime validator both
|
||||||
|
// derive from it, so adding a page is a one-line change (no parallel allowlists to keep in
|
||||||
|
// sync). "feed" is the default/fallback.
|
||||||
|
export const PAGES = [
|
||||||
|
"feed",
|
||||||
|
"channels",
|
||||||
|
"stats",
|
||||||
|
"playlists",
|
||||||
|
"settings",
|
||||||
|
"scheduler",
|
||||||
|
"notifications",
|
||||||
|
] as const;
|
||||||
|
|
||||||
|
export type Page = (typeof PAGES)[number];
|
||||||
|
|
||||||
|
/** Narrow an arbitrary string to a known Page (else null). */
|
||||||
|
export function isPage(p: string | null | undefined): p is Page {
|
||||||
|
return !!p && (PAGES as readonly string[]).includes(p);
|
||||||
|
}
|
||||||
|
|
||||||
export function readPage(): Page {
|
export function readPage(): Page {
|
||||||
const p = new URLSearchParams(window.location.search).get("page");
|
const p = new URLSearchParams(window.location.search).get("page");
|
||||||
return p === "channels" ||
|
return isPage(p) ? p : "feed";
|
||||||
p === "stats" ||
|
|
||||||
p === "playlists" ||
|
|
||||||
p === "settings" ||
|
|
||||||
p === "scheduler"
|
|
||||||
? p
|
|
||||||
: "feed";
|
|
||||||
}
|
}
|
||||||
|
|
||||||
/** Build a shareable absolute URL that reproduces the current filters (+ page). Used by the
|
/** Build a shareable absolute URL that reproduces the current filters (+ page). Used by the
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue