Skip to content

Conversation

@juliusknorr
Copy link
Member

@juliusknorr juliusknorr commented Sep 17, 2024

  • Introduce a ServerVersion class with properly typed methods for getting different version parts and strings
  • Remove and migrate any usages of OC_Util to the new API

Next step could be to deprecated related OCP\Util methods.

@juliusknorr juliusknorr added this to the Nextcloud 31 milestone Sep 17, 2024
@juliusknorr juliusknorr force-pushed the clean/version-ocp branch 5 times, most recently from 6c038da to 2e206ca Compare September 18, 2024 13:06
@juliusknorr juliusknorr requested review from a team, come-nc, icewind1991, nickvergessen and skjnldsv and removed request for a team September 18, 2024 18:22
@juliusknorr juliusknorr force-pushed the clean/version-ocp branch 2 times, most recently from 880cd95 to 31f4e27 Compare September 19, 2024 18:27
@juliusknorr juliusknorr merged commit fd576ce into master Sep 20, 2024
@juliusknorr juliusknorr deleted the clean/version-ocp branch September 20, 2024 16:14
@nickvergessen
Copy link
Member

Since this was defacto-standard, please document the new API and mention the removal in the docs

@nickvergessen nickvergessen added the pending documentation This pull request needs an associated documentation update label Sep 26, 2024
@skjnldsv skjnldsv mentioned this pull request Jan 7, 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.

5 participants