Skip to content

Filter out service worker based content from the online library #887

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

Merged
merged 1 commit into from
Sep 25, 2022

Conversation

kelson42
Copy link
Collaborator

Fixes #780 and superseeds #870

@kelson42
Copy link
Collaborator Author

@mgautierfr @veloman-yunkan Can I merge this?

Copy link
Collaborator

@juuz0 juuz0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unfortunately, doesn't work for me, file still shows up in my library.
This is the file
https://drive.google.com/file/d/1e_sSHD86NI6u7A-LetWW1C6Ap16w1p2d/view?usp=sharing

@kelson42
Copy link
Collaborator Author

Yes, this is for the online library only. To me this is good enough. If someone wants really to load one, and use it with HTTP daemon, then it should be possible. This is why I have talked of OPDS stream in the ticket, but thiscwad not super specific indeed.

@kelson42 kelson42 merged commit fde6d1f into master Sep 25, 2022
@kelson42 kelson42 deleted the no-sw-zim branch September 25, 2022 18:57
mgautierfr added a commit that referenced this pull request Nov 30, 2022
 * Fix the AppImage packaging. Now published AppImages work correctly on recent
   distrubution (@mgautierfr #905)
 * Improve zim file picker (@kelson42 #886)
 * Do not show ServiceWorker zim file from the remote catalog (@kelson42 #887)
@mgautierfr mgautierfr mentioned this pull request Nov 30, 2022
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

Successfully merging this pull request may close these issues.

Zim files requiring Service Workers should be hidden from the library if SW are not available
3 participants