Skip to content

Conversation

@provokateurin
Copy link
Member

No description provided.

@provokateurin provokateurin added this to the Nextcloud 32 milestone Jul 15, 2025
@provokateurin provokateurin requested a review from a team as a code owner July 15, 2025 10:59
@provokateurin provokateurin requested review from Altahrim, ArtificialOwl and leftybournes and removed request for a team July 15, 2025 10:59
@provokateurin provokateurin added 3. to review Waiting for reviews technical debt pending documentation This pull request needs an associated documentation update labels Jul 15, 2025
@provokateurin provokateurin enabled auto-merge July 15, 2025 12:22
Copy link
Contributor

@salmart-dev salmart-dev left a comment

Choose a reason for hiding this comment

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

LGTM!

One of the constants was deprecated in 2021, but I guess this can be ignored.

@provokateurin
Copy link
Member Author

One of the constants was deprecated in 2021, but I guess this can be ignored.

Anything deprecated can be removed after 9 releases/3 years, so this is fine.

Signed-off-by: provokateurin <kate@provokateurin.de>
@provokateurin provokateurin force-pushed the chore/share-constants branch from 40b5f52 to 3c7fce3 Compare July 29, 2025 08:39
@provokateurin
Copy link
Member Author

Rebased to fix Cypress.

@kesselb kesselb disabled auto-merge July 29, 2025 11:31
@kesselb kesselb merged commit b8ad6d0 into master Jul 29, 2025
224 of 238 checks passed
@kesselb kesselb deleted the chore/share-constants branch July 29, 2025 11:31
@skjnldsv skjnldsv mentioned this pull request Aug 19, 2025
@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

Labels

3. to review Waiting for reviews pending documentation This pull request needs an associated documentation update technical debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants