Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(api): migrate login and logout to API routes #1703

Merged
merged 4 commits into from
Mar 6, 2025

Conversation

nsarrazin
Copy link
Collaborator

@nsarrazin nsarrazin commented Feb 8, 2025

  • Replaced form-based login/logout with fetch-based API routes
  • Updated hooks and components to use new /login and /logout endpoints.

a POST on /login will return the authorization url, the callback function is the same, except the load function is now a GET server endpoint.

- Replaced form-based login/logout with fetch-based API routes
- Updated hooks and components to use new `/api/login` and `/api/logout` endpoints
@nsarrazin nsarrazin added enhancement New feature or request front This issue is related to the front-end of the app. back This issue is related to the Svelte backend or the DB labels Feb 8, 2025
@nsarrazin nsarrazin added this to the Make chat-ui run in tauri milestone Feb 10, 2025
… to `/logout`

remove breaing change to connected apps
@nsarrazin
Copy link
Collaborator Author

cc @cyrilzakka I guess this could have some impact 😄

@nsarrazin nsarrazin changed the title feat(auth): migrate login and logout to API routes feat(api): migrate login and logout to API routes Feb 11, 2025
@nsarrazin nsarrazin changed the base branch from main to feat/api_elysia_setup March 6, 2025 00:35
@nsarrazin nsarrazin merged commit 9a02c6f into feat/api_elysia_setup Mar 6, 2025
4 checks passed
@nsarrazin nsarrazin deleted the feat/login_api_refactor branch March 6, 2025 00:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
back This issue is related to the Svelte backend or the DB enhancement New feature or request front This issue is related to the front-end of the app.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

src/routes/logout/+page.server.ts src/routes/login/+page.server.ts
1 participant