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 JSON Feed version 1.1 #400

Closed
1 of 3 tasks
przemoc opened this issue Oct 24, 2021 · 2 comments
Closed
1 of 3 tasks

Support JSON Feed version 1.1 #400

przemoc opened this issue Oct 24, 2021 · 2 comments
Assignees

Comments

@przemoc
Copy link

przemoc commented Oct 24, 2021

Requested Feature: Support JSON Feed version 1.1

Changes in Version 1.1 — 8/7/2020:

  • Updated to use more specific application/feed+json MIME type.
  • Added authors field to allow more than 1 author per feed or item. Deprecated author.
  • Added language field.
  • Clarified that id should be a string.
  • Updated links to JSON and other specs.

Related Area: Plugin

Do you want to contribute this yourself as a pull request?

  • Yes, I have already written code for it (link if available and feasible)
  • Yes, I don’t have code ready yet (that’s okay!)
  • No (that’s okay too!)

Does this feature affect backwards compatibility? If yes, in what way?
It replaces author field (object) with authors field (array of objects).
For backwards compatibility I believe both could be provided.

Rationale and full description: (why should it be added to Nikola?)
Currently jsonfeed plugin supports only version 1.0.
It's better to support latest version of the standard, i.e. 1.1 (which is 1 year old already) for better interoperability in the ecosystem.

@Kwpolska Kwpolska transferred this issue from getnikola/nikola Oct 24, 2021
@Kwpolska Kwpolska self-assigned this Oct 24, 2021
@Kwpolska
Copy link
Member

Those changes were fairly minor and easy to implement, so I went ahead and did them.

(I moved the issue to the plugin repo, since it fits better here.)

@przemoc
Copy link
Author

przemoc commented Oct 24, 2021

Sorry for misplacing the issue in main repo instead of plugins repo.
Thank you for addressing the request so quickly!

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