-
Notifications
You must be signed in to change notification settings - Fork 21
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
Please add support for Nextcloud 24 #44
Comments
Just gonna echo this, I'd love to see NC 24 support as well. |
metoo |
So do I |
#MeToo |
May 20th this was reported and today its June 24th. I hope this great app did not stop being updated. One of the best Nextcloud apps. |
I would love to see this for NC 24 as well. Just started using Nextcloud and was looking for a decent ebook reader app and found this is unfortunately not compatible anymore. I really hope it still get's updated. |
For anyone that doesn't mind getting their hands dirty #45 seems to fix the issue. |
@Emiliaaah If it doesn't get merged, maybe you could put a fork on the nextcloud store? |
@LeifAndersen I wouldn't mind being the person doing that. The only thing is I have no experience maintaining nextcloud apps. If I'd only be moderating and merging PR's I wouldn't mind doing that. Otherwise someone else might be a better fit. |
Seems to work for me too. Just change "4096" to "4000" in the file apps/epubreader/lib/Migration/Version010404Date20201030180941.php at line 73 and line 119 |
@Emiliaaah Ya, I don't have any experience either. :( But it does seem like one of us needs to do it, as this app doesn't seem to be maintained any more. :( |
In case anyone can't find this file, try |
Just curious if anyone is unable to use this in Next Cloud 24 with postgresql. I don't see any errors at all, it just doesn't appear to work. I've tried the various fixes to no avail. I have enabled Having an app like this looks like it would be really great. Please let me know if you can help me find out what's not working. I don't know much about php. |
In one of my comments above, I mentioned a PR with the solution to the only issue I've run into myself when trying to install this app on NC 24. |
Just in case it wasn't noticed - Nextcloud improved policy regarding changes in core to ease app developers in maintaining compatibility with newer Nextcloud versions: |
This worked for me! If anyone is having trouble finding the file, try running this from your nextcloud device |
Could perhaps someone add this to the appstore version? |
That's the problem, since the maintainer doesn't seem to be active there's no one that can update the version in the app store. |
After the Upgrade to NC 24 the app is not compatible anymore.
The text was updated successfully, but these errors were encountered: