Skip to content
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.

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

Add nextcloud 28 support #311

Merged
merged 1 commit into from
Feb 5, 2024
Merged

Add nextcloud 28 support #311

merged 1 commit into from
Feb 5, 2024

Conversation

WhoAmI0501
Copy link
Contributor

@WhoAmI0501 WhoAmI0501 commented Jan 12, 2024

Since no one else is doing it, i implemented some fixes for Nextcloud 28. Works on my machine.

May solves #310 and #306

Want to try it?

sudo -i  # or su - root or whatever you need to do to become root on your box
apt update
apt install make cpio rpm2cpio curl git
cd /tmp
git clone https://github.com/WhoAmI0501/documentserver_community.git
cd documentserver_community
make
cd ..
tar cf documentserver_community.tar documentserver_community/3rdparty/ documentserver_community/appinfo/ documentserver_community/img/ documentserver_community/js/ documentserver_community/lib/ documentserver_community/README.md
cd /var/www/html/nextcloud/apps/  # or where-ever your nextcloud is installed
mv documentserver_community documentserver_community_old
tar xf /tmp/documentserver_community.tar
chown -R www-data:www-data documentserver_community  # or whatever user/group you're using
cd ..
sudo -u www-data php occ upgrade

Signed-off-by: WhoAmI0501 <[email protected]>
This was referenced Jan 12, 2024
@menschentier
Copy link

works perfectly - thank you!

@phillipp
Copy link

@icewind1991 Could you please merge the pull request?

@rsedlmeier
Copy link

rsedlmeier commented Feb 2, 2024

Wounderful, it works and it feels like an performance improvement.

1000 Thx
Cheers Roland

@chadek
Copy link
Collaborator

chadek commented Feb 2, 2024

Sorry did not saw this before, it's been a while since I did not touch my nextcloud instance. Looks good to me, the only thing I see is that for nextcloud 28 support you only need to bump the version number right? You should then remove the user limit update from this pr to track real changes. Thanks for the contribution!

@chadek chadek merged commit b13e4f6 into nextcloud:master Feb 5, 2024
1 check passed
@WhoAmI0501
Copy link
Contributor Author

Sorry for not responding anymore here, was quite busy. Thanks for your reaction and implementing the fix upstream. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants