Skip to content

chore!: remove legacy functions from OC_Helper deprecated before v10 #52912

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
susnux merged 3 commits into from
May 17, 2025
Merged

chore!: remove legacy functions from OC_Helper deprecated before v10 #52912

susnux merged 3 commits into from
May 17, 2025

Conversation

@susnux
Copy link
Contributor

@susnux susnux commented May 16, 2025 •
edited
Loading

Summary

Quite old stuff I could not find any using app for.
All of it has alternatives, so should be safe.

Checklist

@susnux susnux added this to the Nextcloud 32 milestone May 16, 2025
@susnux susnux requested a review from a team as a code owner May 16, 2025 15:01
@susnux susnux added 3. to review Waiting for reviews technical debt labels May 16, 2025
@susnux susnux requested review from artonge, come-nc and skjnldsv and removed request for a team May 16, 2025 15:01
@susnux susnux added pending documentation This pull request needs an associated documentation update ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) labels May 16, 2025
@susnux susnux requested a review from provokateurin May 16, 2025 15:01
@susnux susnux force-pushed the chore/remove-deprecated-oc-helper branch from 07e9541 to 494a079 Compare May 16, 2025 21:10
@susnux susnux merged commit f47c15f into master May 17, 2025
192 of 212 checks passed
@susnux susnux deleted the chore/remove-deprecated-oc-helper branch May 17, 2025 08:55
@susnux susnux mentioned this pull request May 17, 2025
Merged
susnux added a commit to nextcloud/documentation that referenced this pull request May 17, 2025
@susnux
chore(developer): document removals in OC_Helper
2b74f63 
- for nextcloud/server#52912

Signed-off-by: Ferdinand Thiessen <[email protected]>
@susnux susnux removed the pending documentation This pull request needs an associated documentation update label May 17, 2025
@nabim777 nabim777 mentioned this pull request May 19, 2025
Merged
8 tasks
@skjnldsv skjnldsv mentioned this pull request Aug 19, 2025
Merged
@skjnldsv skjnldsv modified the milestones: Nextcloud 32, Nextcloud 33 Sep 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@provokateurin provokateurin provokateurin approved these changes

@come-nc come-nc come-nc approved these changes

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

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

Assignees

No one assigned

Labels

3. to review Waiting for reviews ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) technical debt

Projects

None yet

Milestone

Nextcloud 32

Development

Successfully merging this pull request may close these issues.

5 participants

@susnux @provokateurin @come-nc @skjnldsv