Skip to content

Conversation

@VaiTon
Copy link
Member

@VaiTon VaiTon commented Jun 28, 2025

No description provided.

VaiTon added 6 commits June 28, 2025 15:58
- Add GitHub OAuth login flow with PKCE (api/login/github) - Store
GitHub access/refresh tokens in secure cookies - Add UserBadge component
for login/logout UI - Add hooks.server.ts to manage session tokens and
refresh - Add server-side sessionStore for token management - Add
githubApi utility for GitHub API calls - Add upload page and logic to
create PRs via GitHub App - Update Breadcrumbs to show upload button -
Remove old upld.ts and legacy login logic - Show user badge in layouts
and dashboards - Update dependencies: add octokit, @octokit/auth-app,
@iconify-json/mdi - Update config submodule - Add login page and logout
endpoint
Users are now redirected back to their intended page after login.
@vercel
Copy link

vercel bot commented Jun 28, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
risorse ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 28, 2025 4:58pm

@ali-benny
Copy link
Member

ali-benny commented Jun 29, 2025

Non mi sembra funzionare, quando vai su "login with Github" da 500

Comunque era già praticamente finito sulla branch feat/upld

@VaiTon
Copy link
Member Author

VaiTon commented Jun 29, 2025

Non funziona perché non sono impostati i segreti sul deployment. Purtroppo non posso farlo perché altrimenti tutte le PR avrebbero accesso ai segreti di prod.

Comunque era già praticamente finito sulla branch feat/upld

Si, mi ricordo, ma ho rivisto un po' tutto il procedimento ed il fatto di avere backend e frontend complica di non poco la situazione. Nel caso più generale dovremmo avere un backend con delle utenze e poi gestire OAuth come è stato pensato (standard flow). In questo modo semplifichiamo il tutto avendo tutto qui (ma dipendendo fortemente dalle API di GitHub)

@samuelemusiani
Copy link
Member

La lascio a @ali-benny

@ali-benny
Copy link
Member

@VaiTon Ho provato in locale non ripeto, non sembra andare

Per quanto riguarda le navbar, propongo di usare lo stesso stile che avevo fatto in polleg [se verrà mergiato]

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.

4 participants