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

nextCloud 15? #114

Open
BloodyIron opened this issue Jan 11, 2019 · 15 comments
Open

nextCloud 15? #114

BloodyIron opened this issue Jan 11, 2019 · 15 comments

Comments

@BloodyIron
Copy link

Any chance we can get this compatible with nextCloud 15? Please? Without having to fork this? :/

@4oo4
Copy link

4oo4 commented Jan 25, 2019

@BloodyIron All you should have to do is to edit appinfo/info.xml of both apps to have the max-version for Nextcloud set to 15.0, and they both seem to work:

    <dependencies>
        <nextcloud min-version="8.1" max-version="15.0"/>
                <database>pgsql</database>
                <database>sqlite</database>
                <database>mysql</database>
    </dependencies>

@BloodyIron
Copy link
Author

If it's such little work, why can't the releaser dev address it then?

@rlitman
Copy link

rlitman commented Feb 20, 2019

Because when I try that on my install, it causes Nextcloud to die with an Internal Server Error.
I tried the patch from here:
but Nextcloud complained automatically disabled the app, because it "made Nextcloud unstable"

@BloodyIron
Copy link
Author

Sounds like it's not that little work then...

@ghost
Copy link

ghost commented Mar 19, 2019

Part of having a Nextcloud is that you manage it yourself. In the time it took you to make this issue, you could have filed a PR and/or already fixed it for yourself. If you don't like the developer's response time, then fork it.

@4oo4 Not every (Nextcloud|GitHub) user is such a brilliant developer like you and can fix it for himself.
There's no simple change for files_reader from 13 to 15 in info.xml because the error message Error: This app can not be enabled because it makes the server unstable pops up.

Not every Nextcloud admin knows which fork of this app works for now. Apart from the signing certificate.

After more than one year without any change and support it seems to me that this app is no longer maintained. The author should drop a line in the README.md file and the repo header.

@BloodyIron
Copy link
Author

:/ is there any other way to read epub or other such formats in nextCloud then?

@ghost
Copy link

ghost commented Mar 19, 2019

IMHO with OnlyOffice only, but for that you have to install a Docker container and so this solution for Shared Hosting is as good as not usable.

@4oo4
Copy link

4oo4 commented Mar 20, 2019

@reiner-text I agree that the README.md needs to be updated to reflect that it's unmaintained, but I deleted the comment you're quoting because it was rude and unhelpful. A part of self-hosting does involve personal responsibility (and encourages tinkering like that), but you're right that my hack definitely isn't a replacement for the maintenance that this needs, it was just something that happened to work for me.

I'm not a developer, just a self-hosting nerd like everyone else, and I'm sorry for bringing that garbage into this thread.

@BloodyIron
Copy link
Author

I hope we can find some way to bring this back, because I think nextCloud as a personal library, for epub and so much more, is just a really awesome idea. And I'm really bummed out about it not working :(

@BloodyIron
Copy link
Author

Bump? Anyone prepared to fork?

@BloodyIron
Copy link
Author

Tried the manual override to xml for v15 and it made my nextCloud completely unstable, had to manually disable the app with OCC CLI.

:(

@ghost
Copy link

ghost commented Jul 4, 2019

It's an abandoned app, try Ebook reader instead. 😉

@rlitman
Copy link

rlitman commented Jul 4, 2019

Thanks, just did. Ebook reader is a little clunky, but for a 0.0.1 release, I would say that it has exceeded my expectations, and works well enough.

@ghost
Copy link

ghost commented Jul 4, 2019

It's a good replacement for ebook displaying.

When this application is enabled publicly shared ebook documents will also get shown in the ebook reader instead of only showing a single static snapshot of the document.

This doesn't work for me, unfortunately.

@rlitman
Copy link

rlitman commented Jul 4, 2019

I tried it with a few epub files. There is no way to flip pages with the mouse, but you can get to the table of contents by clicking on the hamburger, and you can use your cursor keys to flip pages. I am not sure how that would work in a mobile interface.

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

3 participants