Skip to content

refactor(dav): remove use of IAppContainer in favor of ContainerInterface #55953

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.

Sign up for GitHub

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

Jump to bottom
Merged
kesselb merged 1 commit into from
Oct 23, 2025
Merged

refactor(dav): remove use of IAppContainer in favor of ContainerInterface #55953

kesselb merged 1 commit into from
Oct 23, 2025

Conversation

@tcitworld
Copy link
Member

@tcitworld tcitworld commented Oct 23, 2025 •
edited by st3iny
Loading

Checklist

@tcitworld tcitworld added this to the Nextcloud 33 milestone Oct 23, 2025
@tcitworld tcitworld requested a review from a team as a code owner October 23, 2025 07:16
@tcitworld tcitworld added the 2. developing Work in progress label Oct 23, 2025
@tcitworld tcitworld requested review from ArtificialOwl, nfebe and yemkareems and removed request for a team October 23, 2025 07:16
@tcitworld
refactor(dav): remove use of IAppContainer in favor of ContainerInter…
9d6c5a8 
…face

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
@tcitworld tcitworld force-pushed the remove-use-of-iappcontainer branch from 1226133 to 9d6c5a8 Compare October 23, 2025 07:36
st3iny
st3iny approved these changes Oct 23, 2025
View reviewed changes
Copy link
Member

@st3iny st3iny left a comment

Choose a reason for hiding this comment

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

🎉

@tcitworld tcitworld added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Oct 23, 2025
@kesselb kesselb merged commit 43aa883 into master Oct 23, 2025
224 of 235 checks passed
@kesselb kesselb deleted the remove-use-of-iappcontainer branch October 23, 2025 11:30
@nextcloud-bot nextcloud-bot mentioned this pull request Jan 6, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@st3iny st3iny st3iny approved these changes

@CarlSchwan CarlSchwan CarlSchwan approved these changes

@ArtificialOwl ArtificialOwl Awaiting requested review from ArtificialOwl ArtificialOwl is a code owner automatically assigned from nextcloud/server-backend

@nfebe nfebe Awaiting requested review from nfebe nfebe is a code owner automatically assigned from nextcloud/server-backend

@yemkareems yemkareems Awaiting requested review from yemkareems yemkareems is a code owner automatically assigned from nextcloud/server-backend

@provokateurin provokateurin Awaiting requested review from provokateurin

@hamza221 hamza221 Awaiting requested review from hamza221

@SebastianKrupinski SebastianKrupinski Awaiting requested review from SebastianKrupinski

Assignees

No one assigned

Labels

3. to review Waiting for reviews feature: dav technical debt

Projects

None yet

Milestone

Nextcloud 33

Development

Successfully merging this pull request may close these issues.

5 participants

@tcitworld @st3iny @CarlSchwan @kesselb