Skip to content

fix(IRegistrationContext): Use IContainer in registerService factory#58355

Merged
provokateurin merged 4 commits intomasterfrom
fix/ocp/container
Feb 16, 2026
Merged

fix(IRegistrationContext): Use IContainer in registerService factory#58355
provokateurin merged 4 commits intomasterfrom
fix/ocp/container

Conversation

@provokateurin
Copy link
Member

Fixes #58261.

The deprecated methods and IAppContainer and IServer container classes should be removed as well, but it was too much for this quick fix.

@provokateurin provokateurin added this to the Nextcloud 34 milestone Feb 16, 2026
@provokateurin provokateurin requested a review from a team as a code owner February 16, 2026 08:31
@provokateurin provokateurin added bug 3. to review Waiting for reviews labels Feb 16, 2026
@provokateurin provokateurin requested review from ArtificialOwl, come-nc and icewind1991 and removed request for a team February 16, 2026 08:31
@provokateurin provokateurin added the pending documentation This pull request needs an associated documentation update label Feb 16, 2026
@provokateurin provokateurin changed the title fix(IContainer): Fix parameter and return types fix(IRegistrationContext): Use IContainer in registerService factory Feb 16, 2026
Signed-off-by: provokateurin <kate@provokateurin.de>
Signed-off-by: provokateurin <kate@provokateurin.de>
Signed-off-by: provokateurin <kate@provokateurin.de>
Signed-off-by: provokateurin <kate@provokateurin.de>
@provokateurin provokateurin merged commit 35f24bc into master Feb 16, 2026
195 of 200 checks passed
@provokateurin provokateurin deleted the fix/ocp/container branch February 16, 2026 12:36
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 bug pending documentation This pull request needs an associated documentation update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants