Skip to content

refactor: Move alternative login code to a new service - #62351

Open
CarlSchwan wants to merge 2 commits into
masterfrom
carl/alternative-login
Open

refactor: Move alternative login code to a new service#62351
CarlSchwan wants to merge 2 commits into
masterfrom
carl/alternative-login

Conversation

@CarlSchwan

Copy link
Copy Markdown
Member

Summary

And use it in LoginController

TODO

  • ...

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@CarlSchwan CarlSchwan added this to the Nextcloud 35 milestone Jul 20, 2026
@CarlSchwan CarlSchwan self-assigned this Jul 20, 2026
@CarlSchwan
CarlSchwan requested a review from a team as a code owner July 20, 2026 18:19
@CarlSchwan CarlSchwan added 3. to review Waiting for reviews technical debt 🧱 🤔🚀 labels Jul 20, 2026
@CarlSchwan
CarlSchwan requested review from Altahrim, come-nc, icewind1991 and salmart-dev and removed request for a team July 20, 2026 18:19
@CarlSchwan
CarlSchwan force-pushed the carl/alternative-login branch 8 times, most recently from d45fc1b to ce27840 Compare July 21, 2026 13:31
Comment thread core/Controller/LoginController.php Outdated
Comment thread lib/private/Authentication/Login/AlternativeLoginService.php Outdated
And use it in LoginController

Signed-off-by: Carl Schwan <carl@carlschwan.eu>
Signed-off-by: Carl Schwan <carl@carlschwan.eu>
@CarlSchwan
CarlSchwan force-pushed the carl/alternative-login branch from ce27840 to 668e896 Compare August 3, 2026 09:36
@nextcloud-bot nextcloud-bot mentioned this pull request Aug 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews technical debt 🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants