Skip to content

refactor(updater): Move updater to a Controller - #58576

Merged
susnux merged 2 commits into
masterfrom
carl/updater-controller
Apr 29, 2026
Merged

refactor(updater): Move updater to a Controller#58576
susnux merged 2 commits into
masterfrom
carl/updater-controller

Conversation

@CarlSchwan

Copy link
Copy Markdown
Member

Remove yet another .php end point and replace it with a Controller

Checklist

AI (if applicable)

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

@CarlSchwanCarlSchwan self-assigned this Feb 25, 2026
@CarlSchwan
CarlSchwan requested review from a team and Altahrim as code ownersFebruary 25, 2026 16:05
@CarlSchwanCarlSchwan added the 3. to review Waiting for reviews label Feb 25, 2026
@CarlSchwan
CarlSchwan requested a review from a team as a code ownerFebruary 25, 2026 16:05
@CarlSchwanCarlSchwan added the ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) label Feb 25, 2026
@CarlSchwan
CarlSchwanforce-pushed the carl/updater-controller branch from 830c539 to 2555545CompareFebruary 25, 2026 16:05
@CarlSchwan
CarlSchwan requested review from ArtificialOwl, leftybournes, skjnldsv and sorbaugh and removed request for a teamFebruary 25, 2026 16:05
@CarlSchwan
CarlSchwanforce-pushed the carl/updater-controller branch from 2555545 to ac0c6c5CompareFebruary 26, 2026 16:42
Comment threadcore/Controller/UpdateController.php Outdated
Comment threadcore/Controller/UpdateController.php Outdated
Comment threadcore/Controller/UpdateController.php Outdated
Comment threadcore/Controller/UpdateController.php Outdated
Comment threadcore/Controller/UpdateController.php Outdated
Comment threadcore/Controller/UpdateController.php Outdated
Comment threadcore/Controller/UpdateController.php Outdated
Comment threadcore/Controller/UpdateController.php
@CarlSchwan
CarlSchwanforce-pushed the carl/updater-controller branch 3 times, most recently from f5c2e18 to 7eaf2f0CompareMarch 2, 2026 17:58
Comment threadcore/Controller/UpdateController.php
@CarlSchwan

Copy link
Copy Markdown
MemberAuthor

ping, I need another review

@susnuxsusnux left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

Comment threadcore/Controller/UpdateController.php
Comment threadcore/Controller/UpdateController.php
@CarlSchwan
CarlSchwanforce-pushed the carl/updater-controller branch from 7eaf2f0 to b798ecbCompareMarch 6, 2026 12:45
Signed-off-by: Carl Schwan <carlschwan@kde.org>
@susnux
susnuxforce-pushed the carl/updater-controller branch from b798ecb to b53d9a7CompareApril 29, 2026 20:58
@susnux

Copy link
Copy Markdown
Contributor

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@susnux
susnux merged commit c4e2159 into masterApr 29, 2026
276 of 297 checks passed
@susnux
susnux deleted the carl/updater-controller branch April 29, 2026 23:19
This was referenced Apr 30, 2026
This was referenced May 7, 2026
This was referenced May 15, 2026
@nextcloud-botnextcloud-bot mentioned this pull request May 27, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviews♻️ refactorRefactor code (not a bug fix, not a feature just refactoring)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@CarlSchwan@susnux@provokateurin@come-nc@joshtrichards@nextcloud-command