[PR #4660] [MERGED] Extract auth related server handlers from main.ts to server/auth/app.ts #12573

Closed
opened 2026-04-10 21:32:59 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/4660
Author: @joel-jeremy
Created: 3/19/2025
Status: Merged
Merged: 4/9/2025
Merged by: @joel-jeremy

Base: masterHead: server-auth-handlers


📝 Commits (5)

  • c1d6211 Extract auth related server handlers from main.ts to server/auth/app.ts
  • 08f88ad Release notes
  • c9001c8 Update get-openid-config
  • acbf486 Fix typecheck error
  • 1dad9fb Fix lint and typecheck errors

📊 Changes

10 files changed (+46 additions, -398 deletions)

View changed files

📝 packages/desktop-client/src/browser-preload.browser.js (+3 -1)
📝 packages/desktop-client/src/components/ServerContext.tsx (+6 -6)
📝 packages/desktop-client/src/components/manager/subscribe/Login.tsx (+4 -4)
📝 packages/loot-core/src/client/app/appSlice.ts (+1 -1)
📝 packages/loot-core/src/server/auth/app.ts (+18 -5)
📝 packages/loot-core/src/server/main.ts (+4 -297)
📝 packages/loot-core/src/types/handlers.d.ts (+3 -1)
📝 packages/loot-core/src/types/server-handlers.d.ts (+0 -82)
📝 packages/loot-core/typings/window.d.ts (+1 -1)
upcoming-release-notes/4660.md (+6 -0)

📄 Description

Related to https://github.com/actualbudget/actual/issues/1113


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/actualbudget/actual/pull/4660 **Author:** [@joel-jeremy](https://github.com/joel-jeremy) **Created:** 3/19/2025 **Status:** ✅ Merged **Merged:** 4/9/2025 **Merged by:** [@joel-jeremy](https://github.com/joel-jeremy) **Base:** `master` ← **Head:** `server-auth-handlers` --- ### 📝 Commits (5) - [`c1d6211`](https://github.com/actualbudget/actual/commit/c1d6211df541c9a55250e1960e358d93b5d7e45b) Extract auth related server handlers from main.ts to server/auth/app.ts - [`08f88ad`](https://github.com/actualbudget/actual/commit/08f88ad5194358b16328ba5b995464deae7bc202) Release notes - [`c9001c8`](https://github.com/actualbudget/actual/commit/c9001c8da4a571fd67422a401c626ceb6f47215c) Update get-openid-config - [`acbf486`](https://github.com/actualbudget/actual/commit/acbf4863550d2b9fdbee1e52aaff574724594459) Fix typecheck error - [`1dad9fb`](https://github.com/actualbudget/actual/commit/1dad9fb75febc2b8e2c026c2dbdbc435f383f21c) Fix lint and typecheck errors ### 📊 Changes **10 files changed** (+46 additions, -398 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/browser-preload.browser.js` (+3 -1) 📝 `packages/desktop-client/src/components/ServerContext.tsx` (+6 -6) 📝 `packages/desktop-client/src/components/manager/subscribe/Login.tsx` (+4 -4) 📝 `packages/loot-core/src/client/app/appSlice.ts` (+1 -1) 📝 `packages/loot-core/src/server/auth/app.ts` (+18 -5) 📝 `packages/loot-core/src/server/main.ts` (+4 -297) 📝 `packages/loot-core/src/types/handlers.d.ts` (+3 -1) 📝 `packages/loot-core/src/types/server-handlers.d.ts` (+0 -82) 📝 `packages/loot-core/typings/window.d.ts` (+1 -1) ➕ `upcoming-release-notes/4660.md` (+6 -0) </details> ### 📄 Description <!-- Thank you for submitting a pull request! Make sure to follow the instructions to write release notes for your PR — it should only take a minute or two: https://github.com/actualbudget/docs#writing-good-release-notes --> Related to https://github.com/actualbudget/actual/issues/1113 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-10 21:32:59 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#12573