search

omnivore-app / obsidian-omnivore

Obsidian plugin to fetch articles and highlights from Omnivore
MIT License
720 stars 39 forks source link

Reduce number of articles fetched per request #202

Closed Fyssion closed 3 months ago

Fyssion commented 4 months ago

This pull request reduces the number of articles fetched in each request from the Omnivore graphql server. Currently, the number is set to 50. If fetching content is enabled, the graphql server tends to return a 500 server error and causes the plugin to fail to sync articles. This could be a larger problem with the graphql server, but to fix this issue for now, I've lowered the number of articles fetched concurrently to 15. I've tested this PR on my personal Omnivore account.

I believe this is PR resolves #185 and resolves #193. It could be related to #179 as well, but I'm not sure.

If you'd like, I'd also be happy to refactor this change out into a configuration setting (as opposed to just changing the hardcoded value). Let me know if that'd be preferred.

jacksonh commented 3 months ago

This makes sense to me. @sywhb what do you think?

github-actions[bot] commented 2 months ago

:tada: This PR is included in version 1.9.5 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: