scraly / developers-conferences-agenda

This repository list a maximum of tech conferences's date and CFP in order to help conferences organizers, speakers & attendees
https://developers.events/
1.66k stars 409 forks source link

json and atom feeds might be invalid #620

Open jonatan-ivanov opened 1 year ago

jonatan-ivanov commented 1 year ago

Json and atom feeds were introduced in https://github.com/scraly/developers-conferences-agenda/pull/604. When I try to add them on feedly, both fail. The W3C feed validation service also says that the atom feed is invalid.

jespino commented 1 year ago

I can take care of this :)

scraly commented 1 year ago

thanks :)

scraly commented 1 year ago

atom feed is now valid

jonatan-ivanov commented 1 year ago

Thank you!

Unfortunately, I'm still not able to add it to Feedly, as it seems @jespino was not able to do it either.

I have two guesses for that, neither should be an issue I think but we don't really know why feedly does not work:

  1. The validator still prints out a warning for this:
    <link rel="self" href="https://developers.events/feed-events.atom"/>

    Which is not the link to the feed, the feed is under /feed-events.xml.

  2. This also makes me wondering that Atom feed urls usually (not always) use the .atom extension instead of xml (I guess mainly to distinguish between .atom and .rss but I'm not an expert in either of those).
scraly commented 1 year ago

I fixed the warning (in #707) for ATOM feed but we still can't import ATOM and JSON feeds on feedly

scraly commented 1 year ago

By the way, atom feed accepts .atom and .xml

jonatan-ivanov commented 1 year ago

Thank you for fixing it. Yeah, it should not matter, neither of the above should be an issue I think for any readers but now that this is fixed and feedly is still broken, I'm out of ideas.

I tweeted them, let's see if they reply: https://twitter.com/jonatan_ivanov/status/1716279676631716019