search

qw3rtty / neix

neix - a RSS/Atom feed reader for your terminal.
GNU General Public License v3.0
201 stars 12 forks source link

[BUG] feedly opml not imported #20

Open verajosemanuel opened 3 years ago

verajosemanuel commented 3 years ago

Describe the bug unable to import feedly opml

To Reproduce Steps to reproduce the behavior:

  1. Download feedly opml
  2. open neix for importing "neix -i feedly_file.opml"
  3. nothing happened

[neix] 0 feed(s) was imported

Expected behavior Neix should import all feeds from opml file.

Desktop (please complete the following information):

qw3rtty commented 3 years ago

Thanks for opening the issue! I will check it.

qw3rtty commented 3 years ago

@verajosemanuel Can you provide a OPML file from feedly?

PeterDigger commented 3 years ago

Describe the bug Unable to import feedly opml

To Reproduce Steps to reproduce the behavior:

  1. Download feedly opml
  2. Open neix for importing "neix -i feedly_file.opml"
  3. nothing happened

[neix] 0 feed(s) was imported

Expected behavior Neix should import all feeds from opml file.

Desktop (please complete the following information):

OS: Gentoo
Version 0.1.5

Greetings @tomschwarz. Here's the example of OMPL file from my alt account at feedly.

(feedly.opml) [Downloaded on 8 October 2021]

<?xml version="1.0" encoding="UTF-8"?>

<opml version="1.0">
    <head>
        <title>User432's subscriptions in feedly Cloud</title>
    </head>
    <body>
        <outline text="Cooking" title="Cooking">
            <outline type="rss" text="Babish Culinary Universe" title="Babish Culinary Universe" xmlUrl="https://www.youtube.com/feeds/videos.xml?channel_id=UCJHA_jMfCvEnv-3kRjTCQXw" htmlUrl="https://www.youtube.com/channel/UCJHA_jMfCvEnv-3kRjTCQXw"/>
            <outline type="rss" text="Recipes - Binging With Babish" title="Recipes - Binging With Babish" xmlUrl="https://www.bingingwithbabish.com/recipes?format=RSS" htmlUrl="https://www.bingingwithbabish.com/recipes/"/>
            <outline type="rss" text="You Suck At Cooking" title="You Suck At Cooking" xmlUrl="https://www.youtube.com/feeds/videos.xml?channel_id=UCekQr9znsk2vWxBo3YiLq2w" htmlUrl="https://www.youtube.com/channel/UCekQr9znsk2vWxBo3YiLq2w"/>
        </outline>
        <outline text="Tech" title="Tech">
            <outline type="rss" text="Linus Tech Tips" title="Linus Tech Tips" xmlUrl="https://www.youtube.com/feeds/videos.xml?channel_id=UCXuqSBlHAE6Xw-yeJA0Tunw" htmlUrl="https://www.youtube.com/channel/UCXuqSBlHAE6Xw-yeJA0Tunw"/>
            <outline type="rss" text="Marques Brownlee" title="Marques Brownlee" xmlUrl="https://www.youtube.com/feeds/videos.xml?channel_id=UCBJycsmduvYEL83R_U4JriQ" htmlUrl="https://www.youtube.com/channel/UCBJycsmduvYEL83R_U4JriQ"/>
        </outline>
    </body>
</opml>

Each outline tag with title of "Cooking" and "Tech" are the name of the folder in Feedly. The rest listed in the nest are name of the RSS and RSS urls.

Psst IMHO this bug label should be in feature because currently Neix can't adapt to different types of RSS feed config file (optimistically). I do know in the manual has clearly stated "Have to be the same format as the default feed config file." Thank you Tom and others collaborators for maintaining such awesome project.

Edit: I realized there's a wiki page regarding migrating via OPML. Welp, I'll take back the things that I have said for the labelling

qw3rtty commented 3 years ago

@PeterDigger Thanks for your comment! I really appreciate it!

I got the problem, there are outline-tags in outline-tags. The current importer does not check if there are children in the current outline-tag. So this should be checked recursively.

This should be a smaller fix. I will do it as soon as I can. Thanks for your effort to paste in the example!