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

Support full Atom (and RSS) spec #14

Open
dotandimet opened this issue Apr 6, 2018 · 5 comments
Open

Support full Atom (and RSS) spec #14

dotandimet opened this issue Apr 6, 2018 · 5 comments

Comments

@dotandimet
Copy link
Owner

@mdom said:

why not just add type to Mojo::Feed? I would even go so far and add all attributes mentioned in the spec. That shouldn't be too much work and have a negligent runtime cost. And even if we worry about that, we could just use a specialised import list to generate a minimal set of accessors or an extended list. I would love to use atoms expired to save updating feeds if they run their course. :)

I say, +1.

@mdom
Copy link
Contributor

mdom commented Apr 6, 2018

Okay, i take a swing at this, but it could take me some days. Weekends are family time... :)

@mdom
Copy link
Contributor

mdom commented Apr 11, 2018

Okay, it is harder than i expected to create a unified api for all the different feed specifications. It's probably easier to extend Mojo::Feed, write some extensions and see what would be an good fit for Mojo::Feed. I'm not sure if this issue should be kept open; if you don't planning on working on it, maybe we just close it and i open a new issue if I have attributes to add? What do you think?

@dotandimet
Copy link
Owner Author

Yeah, I think that the unified/generic feed API Mojo::Feed currently offers can't shift from being a broadest common denominator to being a superset of all Atom/RSS features. The two formats are sufficiently different that they probably require separate parser classes.

What is Atom's expired attribute you mentioned? Is that part of the standard or some extension?

@mdom
Copy link
Contributor

mdom commented May 16, 2018

Uhm, sorry, i thought i saw some rfc mentioning this extension, but i can't find it in the moment.

@dotandimet
Copy link
Owner Author

Huh, I found expired in the JSON Feed spec. I wonder what sources support that.

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

No branches or pull requests

2 participants