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

Feature Request: JSON Feed support #10

Closed
weierophinney opened this issue Dec 31, 2019 · 6 comments
Closed

Feature Request: JSON Feed support #10

weierophinney opened this issue Dec 31, 2019 · 6 comments

Comments

@weierophinney
Copy link
Member

Would be awesome if Zend Feed supported parsing JSON feeds. Publishing and parsing them is much more straightforward and lightweight compared to RSS and Atom, and adoption is starting to pick up.


Originally posted by @brandonkelly at zendframework/zend-feed#42

@weierophinney
Copy link
Member Author

It would be good, but I'm very wary of a month old standard which all but states that Version 1 may be the only version ever needed. Talk about hubris... The problem with these random drop standards, regardless of who writes them, is their isolation/lack of feedback during development. The issue tracker is littered with basic questions, improvements and problems that seem to firmly point to a Version 2 being needed sooner rather than later.

That said, all wariness aside, it should be relatively easy to add but I would expect support to need an overhaul in time to remain relevant. The spec and Atom/RSS mappings also needs a close reading, and it may be prone to disruption if a major JSONFeed publisher starts hacking in some customisations.

Then again, isn't that all new stuff in a nutshell? :)


Originally posted by @padraic at zendframework/zend-feed#42 (comment)

@weierophinney
Copy link
Member Author

I think major feedreader support it : feebin.com, inoreader.com etc. not sure how much work needed, but will be great if we add json feed support.


Originally posted by @rajjanorkar at zendframework/zend-feed#42 (comment)

@froschdesign
Copy link
Member

The problem so far has been that there was no schema which can be used for validation. But now there are schemas for version 1.0 and 1.1 of "JSON Feed": https://github.com/sonicdoe/jsonfeed-schema

I already created a prototype for reading a JSON feed but for the writing part we need to split the Laminas\Feed\Writer\Renderer\RendererInterface because it's based on XML output:

/**
* Save feed and/or entry to XML and return string
*
* @return string
*/
public function saveXml();
/**
* Get DOM document
*
* @return DOMDocument
*/
public function getDomDocument();
/**
* Get document element from DOM
*
* @return DOMElement
*/
public function getElement();

@dvikan
Copy link

dvikan commented Oct 16, 2023

+1

1 similar comment
@rajpt
Copy link

rajpt commented Oct 17, 2023

+1

@Ocramius
Copy link
Member

Ocramius commented Oct 17, 2023

Ah yep, closing+locking until I see real traction on that spec.

@Ocramius Ocramius self-assigned this Oct 17, 2023
@Ocramius Ocramius added the Won't Fix This will not be worked on label Oct 17, 2023
@laminas laminas locked as resolved and limited conversation to collaborators Oct 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants