Skip to content

Conversation

@susnux
Copy link
Contributor

@susnux susnux commented Oct 29, 2025

Summary

Both are deprecated since Nextcloud 19.

Checklist

@susnux susnux added this to the Nextcloud 33 milestone Oct 29, 2025
@susnux susnux self-assigned this Oct 29, 2025
@susnux susnux requested a review from a team as a code owner October 29, 2025 22:23
@susnux susnux added 3. to review Waiting for reviews technical debt pending documentation This pull request needs an associated documentation update labels Oct 29, 2025
@susnux susnux requested review from artonge and szaimen and removed request for a team October 29, 2025 22:23
@susnux susnux added the ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) label Oct 29, 2025
@susnux susnux requested a review from sorbaugh October 29, 2025 22:23
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.

🐘

Both are deprecated since Nextcloud 19.
- For `get`, if really needed, use https://lodash.com/docs#get
- And for `set` https://lodash.com/docs#set

Signed-off-by: Ferdinand Thiessen <[email protected]>
@susnux susnux force-pushed the chore/remove-oc-set-get branch from 4dc2763 to bea30fc Compare October 30, 2025 21:06
@susnux
Copy link
Contributor Author

susnux commented Oct 30, 2025

/compile

Signed-off-by: nextcloud-command <[email protected]>
@susnux
Copy link
Contributor Author

susnux commented Oct 30, 2025

Documentation in nextcloud/documentation#13827

@susnux susnux removed the pending documentation This pull request needs an associated documentation update label Oct 30, 2025
@susnux susnux merged commit f4e8cd1 into master Oct 30, 2025
135 of 137 checks passed
@susnux susnux deleted the chore/remove-oc-set-get branch October 30, 2025 22:13
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.

6 participants