epubreader icon indicating copy to clipboard operation
epubreader copied to clipboard

Please add support for Nextcloud 24

Open SirKubiac opened this issue 2 years ago • 18 comments

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

SirKubiac avatar May 20 '22 06:05 SirKubiac

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

Emiliaaah avatar May 25 '22 01:05 Emiliaaah

metoo

xinstein avatar May 25 '22 02:05 xinstein

So do I

Thomas-Lupo avatar May 25 '22 11:05 Thomas-Lupo

#MeToo

DeRaafMedia avatar Jun 12 '22 19:06 DeRaafMedia

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.

Mangleuz avatar Jun 24 '22 18:06 Mangleuz

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.

DerBoki avatar Jul 06 '22 22:07 DerBoki

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

Emiliaaah avatar Jul 07 '22 22:07 Emiliaaah

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

LeifAndersen avatar Jul 08 '22 03:07 LeifAndersen

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

Emiliaaah avatar Jul 08 '22 18:07 Emiliaaah

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

Sanookmakmak avatar Jul 15 '22 15:07 Sanookmakmak

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

LeifAndersen avatar Jul 25 '22 20:07 LeifAndersen

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

tanpro260196 avatar Jul 30 '22 17:07 tanpro260196

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.

habys avatar Aug 05 '22 13:08 habys

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.

Emiliaaah avatar Aug 14 '22 14:08 Emiliaaah

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

p-bo avatar Dec 01 '22 14:12 p-bo

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"

devbyaccident avatar Dec 23 '22 19:12 devbyaccident

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?

Mangleuz avatar Dec 23 '22 19:12 Mangleuz

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.

Emiliaaah avatar Dec 27 '22 20:12 Emiliaaah