Skip to content

Conversation

@susnux
Copy link
Contributor

@susnux susnux commented Jan 16, 2026

Summary

This API is not deprecated and still used in many places.
So make the class jQuery free to be able to drop it as a dependency.
Also added some unit tests for it.

Checklist

@susnux susnux added this to the Nextcloud 33 milestone Jan 16, 2026
@susnux susnux requested a review from a team as a code owner January 16, 2026 11:09
@susnux susnux added 3. to review Waiting for reviews technical debt labels Jan 16, 2026
@susnux susnux requested review from artonge and removed request for a team January 16, 2026 11:09
@susnux susnux added the ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) label Jan 16, 2026
Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

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

🐘

Make the class jQuery free to be able to drop it as a dependency.
Also added some unit tests for it.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux merged commit c9a7654 into master Jan 16, 2026
146 of 150 checks passed
@susnux susnux deleted the refactor/oc-msg branch January 16, 2026 17:29
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 ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) technical debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants