e-alfred / epubreader

EPUB/CBR/PDF reader app for Nextcloud
GNU Affero General Public License v3.0
79 stars 21 forks source link

Please add support for Nextcloud 24 #44

Open SirKubiac opened 2 years ago

SirKubiac commented 2 years ago

After the Upgrade to NC 24 the app is not compatible anymore.

Emiliaaah commented 2 years ago

Just gonna echo this, I'd love to see NC 24 support as well.

xinstein commented 2 years ago

metoo

Thomas-Lupo commented 2 years ago

So do I

DeRaafMedia commented 2 years ago

MeToo

Mangleuz commented 2 years ago

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.

DerBoki commented 2 years ago

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.

Emiliaaah commented 2 years ago

For anyone that doesn't mind getting their hands dirty #45 seems to fix the issue.

LeifAndersen commented 2 years ago

@Emiliaaah If it doesn't get merged, maybe you could put a fork on the nextcloud store?

Emiliaaah commented 2 years ago

@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.

Sanookmakmak commented 2 years ago

For anyone that doesn't mind getting their hands dirty #45 seems to fix the issue.

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

LeifAndersen commented 2 years ago

@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. :(

tanpro260196 commented 1 year ago

Just change "4096" to "4000" in the file apps/epubreader/lib/Migration/Version010404Date20201030180941.php

In case anyone can't find this file, try custom_apps instead of apps

habys commented 1 year ago

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 'debug' => true in my config.php and still can not see an error message.

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.

Emiliaaah commented 1 year ago

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.

p-bo commented 1 year ago

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:

https://help.nextcloud.com/t/new-process-for-documenting-core-code-changes-that-affect-app-developers/149828?u=p-bo

devbyaccident commented 1 year ago

Just change "4096" to "4000" in the file apps/epubreader/lib/Migration/Version010404Date20201030180941.php

This worked for me! If anyone is having trouble finding the file, try running this from your nextcloud device sudo find / -name "Version010404Date20201030180941.php"

Mangleuz commented 1 year ago

Just change "4096" to "4000" in the file apps/epubreader/lib/Migration/Version010404Date20201030180941.php

This worked for me! If anyone is having trouble finding the file, try running this from your nextcloud device sudo find / -name "Version010404Date20201030180941.php"

Could perhaps someone add this to the appstore version?

Emiliaaah commented 1 year ago

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. So either someone has to publish this app on the app store with the fix, or the maintainer has to come back and update it.