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

Please add support for Nextcloud 24 #44

Open
SirKubiac opened this issue May 20, 2022 · 18 comments
Open

Please add support for Nextcloud 24 #44

SirKubiac opened this issue May 20, 2022 · 18 comments

Comments

@SirKubiac
Copy link

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

@Emiliaaah
Copy link

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

@xinstein
Copy link

metoo

@Thomas-Lupo
Copy link

So do I

@DeRaafMedia
Copy link

#MeToo

@Mangleuz
Copy link

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
Copy link

DerBoki commented Jul 6, 2022

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
Copy link

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

@LeifAndersen
Copy link

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

@Emiliaaah
Copy link

@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
Copy link

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
Copy link

@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
Copy link

tanpro260196 commented Jul 30, 2022

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
Copy link

habys commented Aug 5, 2022

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
Copy link

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
Copy link

p-bo commented Dec 1, 2022

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
Copy link

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
Copy link

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
Copy link

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.

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

No branches or pull requests