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

handle <link href="" />' tag in ATOM #87

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

Conversation

pi-chan
Copy link

@pi-chan pi-chan commented Feb 25, 2014

Hi, I noticed that MWFeedItem's link property is nil, when link tag is like <link href="" />.
So changed implementation for processing link tag for ATOM

@pi-chan
Copy link
Author

pi-chan commented Mar 27, 2014

I found that there are some ATOM feed with <link> tag with no rel attribute.
So I added new else if clause to handle them.

xoyip
Sent with Airmail

On 2014年3月27日 at 21:41:57, Lukas Herbst ([email protected]) wrote:

Hey can you tell me how you changed the implementation? :)


Reply to this email directly or view it on GitHub:
#87 (comment)

@HerbstLukas
Copy link

perfekt, worked fine for me :) thanks!

@taylor-paul
Copy link

This is years old but I'd still like it to get merged...

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.

3 participants