Skip to content

01a02572 - Restore the team PR table as the first view - #34

Merged
TaprootFreak merged 4 commits into
developfrom
01a02572-restore-pr-table
Aug 26, 2026
Merged

01a02572 - Restore the team PR table as the first view#34
TaprootFreak merged 4 commits into
developfrom
01a02572-restore-pr-table

Conversation

@TaprootFreakAI

Copy link
Copy Markdown
Collaborator

EN:
The signed-in hub website leads with open pull requests for people you can see. GitHub OAuth tokens are stored in sqlite so the table survives hub restarts. A 401 from GitHub drops the token; a 403 does not. Replica tables stay below the PR list.

DE:
Die angemeldete Hub-Website beginnt mit den offenen Pull Requests der sichtbaren Personen. GitHub-OAuth-Tokens liegen in sqlite, damit die Tabelle einen Hub-Restart überlebt. Ein 401 von GitHub löscht das Token; ein 403 nicht. Die Replica-Tabellen bleiben unter der PR-Liste.

Details

Reverses the later removal of GET /api/prs. Scope is read:user repo. The cookie session still holds only the login. Tests cover team filter, assignee, persist across a new hub process, logout, 401 vs 403, and the index column order.

@TaprootFreakAI

Copy link
Copy Markdown
Collaborator Author

EN:
Ready after 3 review passes.
The signed-in hub website leads with the team PR table again; OAuth tokens persist across restarts.

DE:
Bereit nach 3 Review-Durchläufen.
Die angemeldete Hub-Website beginnt wieder mit der Team-PR-Tabelle; OAuth-Tokens überleben Restarts.

Details

pytest is green on 205d481. The branch is mergeable. GitHub tokens live in sqlite, not in the cookie. A 401 deletes only the token that failed; a 403 does not. The replica tables stay below the PR list.

@TaprootFreakAI
TaprootFreakAI marked this pull request as ready for review August 25, 2026 17:14
@TaprootFreak
TaprootFreak merged commit 858f5ac into develop Aug 26, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants