-
Notifications
You must be signed in to change notification settings - Fork 214
Add Miniflux recipe #628
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
base: main
Are you sure you want to change the base?
Add Miniflux recipe #628
Conversation
I don't know if I should add this to webview.js, but it might be a good idea to reload the page occasionally as Miniflux doesn't automatically fetch new posts. Any ideas? |
That is a very good point about the auto-refresh. However, I would not go with it that way, since it means that if a user is browsing something, the reader to refresh automatically on them and they have to search again what they were reading. |
Is there a way to check if the user is currently looking at the service? Could use that to prevent the refresh. |
That could be an idea indeed. At the moment, there is no way to know from the webview of a service if it is in view (though the app knows that). The way I see it (but I don't fully understand the whole codebase yet), you would need to modify the app itself, in order to add a function in |
86c6fd5
to
7ede4a1
Compare
Was able to figure out a way to do this using |
I realized it would make more sense to only reload when the user isn't reading an entry or in settings |
Pre-flight Checklist
Please ensure you've completed all of the following.
Description of Change
Added a recipe for Miniflux RSS reader
https://miniflux.app/