-
Notifications
You must be signed in to change notification settings - Fork 22
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 support to NC 23 #40
Comments
As an administrator of a comic book archive the support on new versions of NC is very important. |
it worked for me today |
@manualkamp - Thanks. It worked for me too. |
Guys, does it save progress for you? I tried NC 22 snap package, as well as 23 run from docker and neither of them saves read position, much to my annoyance |
My PR #42 should fix this (I hope !) |
And it seems it did! :) Thank you for your wonderful contribution to the splendid app. It has minor quirks, but the really worst for all on Nextcloud 24 is that the reader is not compatible and installable from the app repository ;( (Issue is here). Do you perhaps know if this is just a simple meta-tag issue (23->24), or if it might break some of the nice fixes you have done? |
#42 didn't seem to fix the sync for me. |
I'm totally not a Nextcloud App developer, so I can not be sure if it should work (I'm still using Nextcloud 22). I tried to replace the deprecated feature I saw during the fix, so Nextcloud 24 could work. You might test editing the info.xml file to change the supported version. |
Do you have any request to a URL ending with "cursor" while changing pages ? What is HTTP return code for those requests ? |
This app cannot be installed because the following dependencies are not fulfilled:
Server version 21 or lower is required.
The text was updated successfully, but these errors were encountered: