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

feat: import opml handler #3836

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kimlimjustin
Copy link
Contributor

Created OPML import handler, as discussed on #2870, by integrating https://github.com/danmactough/node-opmlparser solution into the packages/import-handler directory.

Yet, please note that due to insufficient up-to-date contributing docs and guidelines, this PR is made by literally reading the whole codebase manually, without actually running the application locally; this PR is only testified by running the mocha testing under the import-handler package.

That being said, please try this PR and please let me know if any errors occurred or if any changes should be made further to complete this pull request,

Thank you and hope that the OPML handler can be part of omnivore soon shortly!

Copy link

vercel bot commented Apr 18, 2024

@kimlimjustin is attempting to deploy a commit to the omnivore Team on Vercel.

A member of the Team first needs to authorize it.

@kallekn
Copy link

kallekn commented Apr 23, 2024

This is a deal breaker. Omnivore looks nice, but I need to be able to import my feeds in order to use it.

@jacksonh
Copy link
Contributor

Hey this is an interesting way of doing it. We actually do have an OPML API also but we haven't exposed it on the web, just because I don't think our handling of lots of RSS feeds is great yet.

@natarslan
Copy link

natarslan commented May 1, 2024

Following this! Would be great to be able to bulk import feeds (opml).

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.

None yet

4 participants